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

[Saleem Abdulrasool via llvm-commits]

compnerd wrote:

> > 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.
> 
> Basically, when using `debuginfologicalview` as a library from a downstream project, `NDEBUG` may be defined if the downstream project is built in Release mode. This results in a different class layout (e.g. for `LVObject`), given that, by the time the `LVObject.h` is parsed in a downstream project, `NDEBUG` may be defined (or not) regardless of whether it was defined (or not) when LLVM was built.
> 
> Thus, `NDEBUG` cannot be relied for anything that affects class layout (at least on public headers). Looking at @dwblaikie and @compnerd comments, perhaps using `LLVM_ENABLE_ABI_BREAKING_CHECKS` (instead of introducing a new `LLVM_BUILD_DEBUG` PP macro) is the way to go 🤔 (?)

Well, `dump` shouldn't be part of the public interface, so removal is a better option. Note the guard - `LLVM_ENABLE_DUMP` is an alternative spelling. Depending on `NDEBUG` is acceptable here. This doesn't change the object layout, but does change the API surface. If the downstream project is attempting to use the `dump` methods, it should be building LLVM with `LLVM_ENABLE_DUMP`.

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