EricWF marked an inline comment as done.
EricWF added inline comments.

================
Comment at: lib/Driver/ToolChains/Clang.cpp:3269
 
-// Disable the verification pass in -asserts builds.
+  const bool IsAssertBuild =
 #ifdef NDEBUG
----------------
lebedev.ri wrote:
> This logic seems sidewards.
> If `NDEBUG` is specified, then it is `assert()`-less build.
> If `NDEBUG` is *not* specified, then `assert()`'s are actually functional.
> 
Woops! Silly me. Thats a dumb mistake and I feel dumb having made it.


https://reviews.llvm.org/D42887



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to