Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Inherent" "impls" save-analysis have empty "value" #60872

Closed
asutherland opened this issue May 16, 2019 · 1 comment
Closed

"Inherent" "impls" save-analysis have empty "value" #60872

asutherland opened this issue May 16, 2019 · 1 comment
Labels
A-save-analysis Area: saving results of analyses such as inference and borrowck results to a file. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue.

Comments

@asutherland
Copy link

For a rust impl block like impl Loader {...}, the current nightly toolchain generates an impls entry that has a span that references the position of "Loader" in the source file, but the value is the empty string "".

As a particular example, https://github.com/mozsearch/mozsearch/blob/7b87459eea1c0a24beb362b7627960260185034b/tests/tests/files/simple.rs#L36 produces the following prettified impls item:

        {
            "span": {
                "byte_start": 748,
                "column_end": 12,
                "column_start": 6,
                "byte_end": 754,
                "line_end": 36,
                "file_name": "simple.rs",
                "line_start": 36
            },
            "parent": null,
            "kind": "Inherent",
            "children": [
                {
                    "index": 27,
                    "krate": 0
                },
                {
                    "index": 28,
                    "krate": 0
                },
                {
                    "index": 29,
                    "krate": 0
                },
                {
                    "index": 30,
                    "krate": 0
                },
                {
                    "index": 31,
                    "krate": 0
                }
            ],
            "value": "",
            "attributes": [],
            "sig": null,
            "id": 1,
            "docs": ""
        }

The context in which I'm concerned about this is for https://github.com/mozsearch/mozsearch. The rust language indexer uses rls to load the analysis data and processes it at a low level. I've hacked around the issue by extracting the contents of the span manually, but it seems useful to include the value.

Suggested label:A-save-analysis

@Xanewok Xanewok added A-save-analysis Area: saving results of analyses such as inference and borrowck results to a file. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue. labels May 16, 2019
@jyn514
Copy link
Member

jyn514 commented Feb 19, 2023

Save-analysis has been removed from the compiler.

@jyn514 jyn514 closed this as not planned Won't fix, can't repro, duplicate, stale Feb 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-save-analysis Area: saving results of analyses such as inference and borrowck results to a file. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants