[Lldb-commits] [PATCH] D58838: Remove tautological #ifdefs

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added a comment.

In D58838#1415637 <https://reviews.llvm.org/D58838#1415637>, @zturner wrote:

> Yea it would be nice if we could remove all of the `LLDB_CONFIGURATION_xxx` macros and just use either the LLVM ones or standard ones such as NDEBUG


I generally agree with this, but we need to decide what to do on a case-by-case basis. So far I found four categories of LLDB_CONFIGURATION_DEBUG

1. assertions that should just be "assert"
2. expensive checks that should be guarded by LLVM_ENABLE_EXPENSIVE_CHECKS instead
3. Consistency checks that assume that the debug info is 100% accurate/complete and that may fail in the real world
4. additional logging

there may be other cases that I haven't found yet.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58838/new/

https://reviews.llvm.org/D58838