[PATCH] D24647: [CUDA] [test-suite] Always enable assert()s in our builds.

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 26 17:38:13 PDT 2016


MatzeB added a subscriber: MatzeB.
MatzeB added a comment.

NDEBUG is currently enable globally in the toplevel CMakeLists.txt, as it is claimed that some tests fail without it (I don't know which ones and it feels like a hack to me to enable it globally). In any case we could move `-DNDEBUG` from the global CMakeLists.txt file to {SingleSource|MultiSource|External}/CMakeLists.txt (or even further down the hierarchy when we know which tests actually need it for correctness).


https://reviews.llvm.org/D24647





More information about the llvm-commits mailing list