[llvm-commits] CVS: llvm/lib/Bytecode/Reader/Reader.cpp

Chris Lattner sabre at nondot.org
Tue Feb 13 22:20:25 PST 2007



Changes in directory llvm/lib/Bytecode/Reader:

Reader.cpp updated: 1.238 -> 1.239
---
Log message:

>From Dan Gohman:

While preparing http://llvm.org/PR1198: http://llvm.org/PR1198  I noticed several asserts
protecting unprepared code from i128 types that weren't actually failing
when they should because they were written as assert("foo") instead of
something like assert(0 && "foo"). This patch fixes all the cases that a
quick grep found.



---
Diffs of the changes:  (+1 -1)

 Reader.cpp |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)


Index: llvm/lib/Bytecode/Reader/Reader.cpp
diff -u llvm/lib/Bytecode/Reader/Reader.cpp:1.238 llvm/lib/Bytecode/Reader/Reader.cpp:1.239
--- llvm/lib/Bytecode/Reader/Reader.cpp:1.238	Tue Feb 13 00:30:42 2007
+++ llvm/lib/Bytecode/Reader/Reader.cpp	Wed Feb 14 00:20:04 2007
@@ -1265,7 +1265,7 @@
       Result = ConstantInt::get(IT, Val);
       if (Handler) Handler->handleConstantValue(Result);
     } else 
-      assert("Integer types > 64 bits not supported");
+      assert(0 && "Integer types > 64 bits not supported");
     break;
   }
   case Type::FloatTyID: {






More information about the llvm-commits mailing list