[llvm-commits] [llvm] r103769 - in /llvm/trunk: lib/CodeGen/LLVMTargetMachine.cpp test/Other/inline-asm-newline-terminator.ll

[Daniel Dunbar]

Author: ddunbar
Date: Fri May 14 02:47:51 2010
New Revision: 103769

URL: http://llvm.org/viewvc/llvm-project?rev=103769&view=rev
Log:
XFAIL the test I added with vg_leak, apparently it is the first and only llc
-filetype=obj test, and -filetype=obj leaks a few objects. Added a FIXME, we
need to sort out the ownership model for the various MC objects.

Modified:
    llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp
    llvm/trunk/test/Other/inline-asm-newline-terminator.ll

Modified: llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp?rev=103769&r1=103768&r2=103769&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp (original)
+++ llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp Fri May 14 02:47:51 2010
@@ -140,6 +140,8 @@
   case CGFT_ObjectFile: {
     // Create the code emitter for the target if it exists.  If not, .o file
     // emission fails.
+    //
+    // FIXME: These are currently leaked.
     MCCodeEmitter *MCE = getTarget().createCodeEmitter(*this, *Context);
     TargetAsmBackend *TAB = getTarget().createAsmBackend(TargetTriple);
     if (MCE == 0 || TAB == 0)

Modified: llvm/trunk/test/Other/inline-asm-newline-terminator.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/inline-asm-newline-terminator.ll?rev=103769&r1=103768&r2=103769&view=diff
==============================================================================
--- llvm/trunk/test/Other/inline-asm-newline-terminator.ll (original)
+++ llvm/trunk/test/Other/inline-asm-newline-terminator.ll Fri May 14 02:47:51 2010
@@ -1,4 +1,5 @@
 ; RUN: llc -filetype=obj -o - < %s
+; XFAIL: vg_leak
 
 ; ModuleID = 't.c'
 target triple = "x86_64-apple-darwin10.0.0"