Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

docs: remove STRIP_FROM_INC_PATH in Doxygen to avoid incorrect source file linking #140114

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
Loading
from

Conversation

TheShubhendra
Copy link

Set STRIP_FROM_INC_PATH to blank in doxygen.cfg.in to preserve full relative paths for source files. This prevents Doxygen from confusing files with the same name in different directories (e.g., lib/IR/Use.cpp vs lib/SandboxIR/Use.cpp) and ensures correct linking in the generated documentation.

Fixes #139512

…linking

Set STRIP_FROM_INC_PATH to blank in doxygen.cfg.in to preserve full relative paths
for source files. This prevents Doxygen from confusing files with the same name
in different directories (e.g., lib/IR/Use.cpp vs lib/SandboxIR/Use.cpp) and
ensures correct linking in the generated documentation.
Copy link

Thank you for submitting a Pull Request (PR) to the LLVM Project!

This PR will be automatically labeled and the relevant teams will be notified.

If you wish to, you can add reviewers by using the "Reviewers" section on this page.

If this is not working for you, it is probably because you do not have write permissions for the repository. In which case you can instead tag reviewers by name in a comment by using @ followed by their GitHub username.

If you have received no comments on your PR for a week, you can request a review by "ping"ing the PR by adding a comment “Ping”. The common courtesy "ping" rate is once a week. Please remember that you are asking for valuable time from other developers.

If you have further questions, they may be answered by the LLVM GitHub User Guide.

You can also ask questions in a comment on this PR, on the LLVM Discord or on the forums.

@TheShubhendra TheShubhendra marked this pull request as ready for review May 15, 2025 18:24
@TheShubhendra
Copy link
Author

@cor3ntin can you please review

@cor3ntin
Copy link
Contributor

Thanks for working on that.

We have several doxygen.cfg.in files in various places in the repo for different subprojects. They should probably be consistent

@TheShubhendra
Copy link
Author

@cor3ntin should I make this change to all the doxygen.cfg.in files in this PR?

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

Successfully merging this pull request may close these issues.

Doxygen links to wrong implementation file
2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.