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

Declarations #98

Open
Bestie123 opened this issue Sep 2, 2022 · 3 comments
Open

Declarations #98

Bestie123 opened this issue Sep 2, 2022 · 3 comments

Comments

@Bestie123
Copy link

Bestie123 commented Sep 2, 2022

source - llvm-project-14.0.6.
the definition is not displayed CodeGenOptions.cpp and so with many files

CodeGenOptions.h
image

CodeGenOptions.cpp
image

for example, in the vs code program, using the c/c++ extension (adds intellisense navigation), the definiton is displayed from CodegenOptions.cpp
image

visual studio also displays CodeGenOptions.cpp
image

in my case, the problem manifests itself in the documentation in the version llvmpoject-14.0.6, but the problem also occurs in the old version in the official example
https://blog.weghos.com/llvm/llvm/clang/lib/Frontend/CodeGenOptions.cpp.html

@ogoffart
Copy link
Contributor

ogoffart commented Sep 2, 2022

The definition here in your screenshot is on the same line.

For .cpp file, it is not there because it is actually a different macro, and since it is defined in a .cpp file, it is considered as private to that file. So no need to add it in the index.

@Bestie123
Copy link
Author

@ogoffart, thank you, indeed, definitions for files.cpps are private. Probably vs code and visual studio just show all the definitions.

@Bestie123
Copy link
Author

This function is a bit inconvenient (when it is not added to indexing) because you will have to mark a link to the .cpp file somewhere if you need to navigate to the file.h, in the file .h the backlink to .cpp will no longer be displayed. This will probably make more sense when exploring a large unknown project with a lot of files and directories. Perhaps it would be possible to add indexing with a note that these are private areas.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants