[Lldb-commits] [PATCH] D38552: LLDB cmake fix: define LLDB_CONFIGURATION_xxx based on the build type
Leonard Mosescu via lldb-commits
lldb-commits at lists.llvm.org
Wed Oct 4 11:33:14 PDT 2017
Identical, no - note the checks are inverted. Definitely uglier, but it's
defaulting to LLDB_CONFIGURATION_DEBUG if CMAKE_BUILD_TYPE is not defined,
With the current defaulting rule I pointed out both versions should be
equivalent. If anything I prefer the 1st version since I like to avoid
negations in conditional expressions, but it's a minor thing and I'll leave
the final choice up to you.
On Wed, Oct 4, 2017 at 11:22 AM, Zachary Turner via Phabricator <
reviews at reviews.llvm.org> wrote:
> zturner added inline comments.
> Comment at: CMakeLists.txt:15
> +# Define the LLDB_CONFIGURATION_xxx matching the build type
> +if( NOT uppercase_CMAKE_BUILD_TYPE STREQUAL "DEBUG" )
> + add_definitions( -DLLDB_CONFIGURATION_RELEASE )
> Isn't this identical to the code before?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits