[PATCH] D28358: [globalisel] Stop requiring -debug/-debug-only=registerbankinfo for assertions.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 5 14:45:42 PST 2017


dsanders added inline comments.


================
Comment at: lib/CodeGen/GlobalISel/RegisterBankInfo.cpp:58
     : RegBanks(RegBanks), NumRegBanks(NumRegBanks) {
-  DEBUG(for (unsigned Idx = 0, End = getNumRegBanks(); Idx != End; ++Idx) {
+#ifdef NDEBUG
+  for (unsigned Idx = 0, End = getNumRegBanks(); Idx != End; ++Idx) {
----------------
qcolombet wrote:
> #ifndef NDEBUG, right?
Yes, you're right. It looks like I accidentally inverted it when I separated it from my WIP.


https://reviews.llvm.org/D28358





More information about the llvm-commits mailing list