[llvm-commits] CVS: llvm/lib/ExecutionEngine/ExecutionEngine.cpp

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



Changes in directory llvm/lib/ExecutionEngine:

ExecutionEngine.cpp updated: 1.105 -> 1.106
---
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:  (+2 -2)

 ExecutionEngine.cpp |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


Index: llvm/lib/ExecutionEngine/ExecutionEngine.cpp
diff -u llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.105 llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.106
--- llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.105	Sat Feb 10 14:35:22 2007
+++ llvm/lib/ExecutionEngine/ExecutionEngine.cpp	Wed Feb 14 00:20:04 2007
@@ -292,7 +292,7 @@
           const_cast<GlobalVariable *>(dyn_cast<GlobalVariable>(GV)))
     EmitGlobalVariable(GVar);
   else
-    assert("Global hasn't had an address allocated yet!");
+    assert(0 && "Global hasn't had an address allocated yet!");
   return state.getGlobalAddressMap(locked)[GV];
 }
 
@@ -425,7 +425,7 @@
     else if (BitWidth <= 64)
       Result.Int64Val = (uint64_t )cast<ConstantInt>(C)->getZExtValue();
     else
-      assert("Integers with > 64-bits not implemented");
+      assert(0 && "Integers with > 64-bits not implemented");
     break;
   }
 






More information about the llvm-commits mailing list