[llvm] [llvm-debuginfo-analyzer] Use `LLVM_BUILD_DEBUG` in class definitions (PR #140265)

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Mon May 19 08:54:46 PDT 2025


compnerd wrote:

> I think we already have a separate macro for this - ABI_BREAKING_CHECKS, perhaps?

The `ABI_BREAKING_CHECKS` is meant for changes to the data structure layout that changes between Asserts and NoAsserts builds IIRC. The dump methods can be enabled/disabled separately (similar to how building with different targets is not considered ABI breaking).

I don't understand the motivation of this change. I realize that `NDEBUG` is a confusing macro, but this is not about a _debug_ build. `NDEBUG` indicates whether assertions are enabled or not.

https://github.com/llvm/llvm-project/pull/140265


More information about the llvm-commits mailing list