[PATCH] cmake: NDEBUG needlessly defined in non-Release builds

Janusz Sobczak janusz.sobczak at mobica.com
Fri Jun 27 08:33:22 PDT 2014


> I'm starting to think we should take this patch.  The effect of landing this patch is to drop support for debug builds without assertions, which is basically a useless configuration anyway.  If we drop this else clause, we basically leave the definition of NDEBUG up to CMake, which is probably fine.

I don't have write permission to the repo. Could you commit this patch?

http://reviews.llvm.org/D4257






More information about the llvm-commits mailing list