<div dir="ltr">Identical, no - note the checks are inverted. Definitely uglier, but it's defaulting to <span style="font-size:12.8px">LLDB_CONFIGURATION_DEBUG if </span><span style="font-size:12.8px">CMAKE_BUILD_TYPE is not defined, right?</span><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">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.</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 4, 2017 at 11:22 AM, Zachary Turner via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">zturner added inline comments.<br>
<br>
<br>
================<br>
Comment at: CMakeLists.txt:15<br>
</span><span class="">+# Define the LLDB_CONFIGURATION_xxx matching the build type<br>
</span>+if( NOT uppercase_CMAKE_BUILD_TYPE STREQUAL "DEBUG" )<br>
+  add_definitions( -DLLDB_CONFIGURATION_RELEASE )<br>
----------------<br>
Isn't this identical to the code before?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D38552" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D38552</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>