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

Adrian Prantl via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 1 14:52:28 PST 2019

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.



More information about the lldb-commits mailing list