[PATCH] D19723: Unify XDEBUG and EXPENSIVE_CHECKS (into the latter), and add an option to the cmake build to enable them.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 29 08:10:43 PDT 2016


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM, nice!


================
Comment at: cmake/modules/HandleLLVMOptions.cmake:48-49
@@ +47,4 @@
+if( LLVM_ENABLE_EXPENSIVE_CHECKS )
+  add_definitions( -DEXPENSIVE_CHECKS )
+  add_definitions( -D_GLIBCXX_DEBUG )
+endif()
----------------
No need for the spaces inside the ()s here AFAIK... but whatever. =]


http://reviews.llvm.org/D19723





More information about the llvm-commits mailing list