[llvm-commits] [llvm] r59677 - /llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp

Evan Cheng evan.cheng at apple.com
Wed Nov 19 15:21:11 PST 2008


Author: evancheng
Date: Wed Nov 19 17:21:11 2008
New Revision: 59677

URL: http://llvm.org/viewvc/llvm-project?rev=59677&view=rev
Log:
Eliminate a compile time warning.

Modified:
    llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp

Modified: llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp?rev=59677&r1=59676&r2=59677&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp Wed Nov 19 17:21:11 2008
@@ -516,10 +516,12 @@
       assert(0 && "psuedo instructions should be removed before code emission");
       break;
     case TargetInstrInfo::INLINEASM: {
-      const char* Value = MI.getOperand(0).getSymbolName();
-      /* We allow inline assembler nodes with empty bodies - they can
-         implicitly define registers, which is ok for JIT. */
-      assert((Value[0] == 0) && "JIT does not support inline asm!\n");
+      // We allow inline assembler nodes with empty bodies - they can
+      // implicitly define registers, which is ok for JIT.
+      if (MI.getOperand(0).getSymbolName()[0]) {
+        assert(0 && "JIT does not support inline asm!\n");
+        abort();
+      }
       break;
     }
     case TargetInstrInfo::DBG_LABEL:





More information about the llvm-commits mailing list