[llvm] [llvm-debuginfo-analyzer] Use `LLVM_BUILD_DEBUG` in class definitions (PR #140265)
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Sun May 18 17:21:37 PDT 2025
dwblaikie wrote:
> > I think we already have a separate macro for this - ABI_BREAKING_CHECKS, perhaps?
>
> Hmmm... unsure; according to [llvm/docs/ProgrammersManual.rst](https://github.com/llvm/llvm-project/blob/main/llvm/docs/ProgrammersManual.rst), this PP macro serves a slightly different purpose, but we could probably also use it here. WDYT, @CarlosAlbertoEnciso?
The difference being a debug build V an asserts-ish build? Yeah, I think the right thing to do would be to make these members enabled/disabled based on ABI_BREAKING_CHECKS..
Oh, the issue is these aren't checks? They're only for debug dumping features? That is a bit awkward. Do they have to be ABI breaking to provide debugging features/could they be implemented in some other way? (& I don't think we enable dump functions based on whether it's a debug build, right? There's some separate macro for that?)
Might be worth a discourse post for discussion
https://github.com/llvm/llvm-project/pull/140265
More information about the llvm-commits
mailing list