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

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 11:35:20 PDT 2016


jlebar added a comment.

In https://reviews.llvm.org/D24647#554137, @MatzeB wrote:

> as long as you just push it the setting down the tree and just not into the CUDA directory, it shouldn't affect anyone.


I'm confused: If pushing the -DNDEBUG down in the llvm directory structure would have no effect on other projects (e.g. clang, lldb), why *would* it have an effect on the CUDA code in the test-suite?  Or are you saying the test-suite is special in this respect?

Sorry for being so clueless here.  /me just wants his asserts.


https://reviews.llvm.org/D24647





More information about the llvm-commits mailing list