Skip to content

Revert "Strip the full path from __FILE__ in the LDBG macro and keep only the filename (#150677)" #11130

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

Conversation

jroelofs
Copy link

@jroelofs jroelofs commented Aug 6, 2025

This reverts commit 5d26e3c.

It breaks the modules build of clang, since every source file has a different version of this macro, which causes
https://green.lab.llvm.org/job/clang-stage2-Rthinlto/ to run out of space. We should probably be using FILE_NAME instead when the host compiler supports it, but until then let's revert-to-green, and un-block some of the bots.

see: llvm#150677
see: llvm#150677 (comment)

rdar://157465825

…only the filename (llvm#150677)"

This reverts commit 5d26e3c.

It breaks the modules build of clang, since every source file has a different
version of this macro, which causes
https://green.lab.llvm.org/job/clang-stage2-Rthinlto/ to run out of space. We
should probably be using __FILE_NAME__ instead when the host compiler supports
it, but until then let's revert-to-green, and un-block some of the bots.

see: llvm#150677
see: llvm#150677 (comment)

rdar://157465825
@jroelofs jroelofs merged commit ada445f into next Aug 6, 2025
@jroelofs jroelofs deleted the jroelofs/revert-5d26e3c227f4b4a1761a8b0001b3165198def479-rdar157465825 branch August 6, 2025 00:18
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

Successfully merging this pull request may close these issues.

1 participant