[llvm-commits] [PATCH] Asserts never trip

Cameron McInally mcinally at cray.com
Tue Jun 7 09:56:59 PDT 2011


Hey guys,

I found a few asserts for illegal paths that should always trip, but
currently never trip:

mcinally/llvm> svn diff
Index: utils/TableGen/FastISelEmitter.cpp
===================================================================
--- utils/TableGen/FastISelEmitter.cpp    (revision 132714)
+++ utils/TableGen/FastISelEmitter.cpp    (working copy)
@@ -278,7 +278,7 @@
       } else if (Operands[i].isFP()) {
         OS << "ConstantFP *f" << i;
       } else {
-        assert("Unknown operand kind!");
+        assert(0 && "Unknown operand kind!");
         abort();
       }
       if (i + 1 != e)
@@ -307,7 +307,7 @@
         OS << "f" << i;
         PrintedArg = true;
       } else {
-        assert("Unknown operand kind!");
+        assert(0 && "Unknown operand kind!");
         abort();
       }
     }
@@ -322,7 +322,7 @@
       } else if (Operands[i].isFP()) {
         OS << "f" << i;
       } else {
-        assert("Unknown operand kind!");
+        assert(0 && "Unknown operand kind!");
         abort();
       }
       if (i + 1 != e)


Tx,
Cameron



More information about the llvm-commits mailing list