[llvm-commits] [llvm] r83822 - in /llvm/trunk/lib: CodeGen/MachineFunctionAnalysis.cpp Target/X86/X86CodeEmitter.cpp

[Chris Lattner]

Author: lattner
Date: Sun Oct 11 23:22:44 2009
New Revision: 83822

URL: http://llvm.org/viewvc/llvm-project?rev=83822&view=rev
Log:
Fix PR5087, patch by Jakub Staszak!

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

Modified: llvm/trunk/lib/CodeGen/MachineFunctionAnalysis.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineFunctionAnalysis.cpp?rev=83822&r1=83821&r2=83822&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/MachineFunctionAnalysis.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachineFunctionAnalysis.cpp Sun Oct 11 23:22:44 2009
@@ -30,6 +30,7 @@
 }
 
 MachineFunctionAnalysis::~MachineFunctionAnalysis() {
+  releaseMemory();
   assert(!MF && "MachineFunctionAnalysis left initialized!");
 }
 

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

==============================================================================
--- llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86CodeEmitter.cpp Sun Oct 11 23:22:44 2009
@@ -587,8 +587,8 @@
     case TargetInstrInfo::INLINEASM:
       // We allow inline assembler nodes with empty bodies - they can
       // implicitly define registers, which is ok for JIT.
-      assert(MI.getOperand(0).getSymbolName()[0] == 0 && 
-             "JIT does not support inline asm!");
+      if (MI.getOperand(0).getSymbolName()[0])
+        llvm_report_error("JIT does not support inline asm!");
       break;
     case TargetInstrInfo::DBG_LABEL:
     case TargetInstrInfo::EH_LABEL: