[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