[llvm-branch-commits] [llvm-branch] r100821 - in /llvm/branches/Apple/Morbo: lib/CodeGen/SelectionDAG/FastISel.cpp lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp test/DebugInfo/2010-01-18-DbgValue.ll

[Devang Patel]

Author: dpatel
Date: Thu Apr  8 17:30:50 2010
New Revision: 100821

URL: http://llvm.org/viewvc/llvm-project?rev=100821&view=rev
Log:
Enable DBG_VALUE in code generator.

Modified:
    llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/FastISel.cpp
    llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
    llvm/branches/Apple/Morbo/test/DebugInfo/2010-01-18-DbgValue.ll

Modified: llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/FastISel.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/FastISel.cpp?rev=100821&r1=100820&r2=100821&view=diff
==============================================================================
--- llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/FastISel.cpp (original)
+++ llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/FastISel.cpp Thu Apr  8 17:30:50 2010
@@ -345,13 +345,10 @@
     
     // Building the map above is target independent.  Generating DBG_VALUE
     // inline is target dependent; do this now.
-    // FIXME: We are not yet ready to handle dbg_value.
-    // (void)TargetSelectInstruction(cast<Instruction>(I));
+    (void)TargetSelectInstruction(cast<Instruction>(I));
     return true;
   }
   case Intrinsic::dbg_value: {
-    // FIXME: We are not yet ready to handle dbg_value.
-    return 0;
     // This requires target support, but right now X86 is the only Fast target.
     DbgValueInst *DI = cast<DbgValueInst>(I);
     const TargetInstrDesc &II = TII.get(TargetOpcode::DBG_VALUE);

Modified: llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp?rev=100821&r1=100820&r2=100821&view=diff
==============================================================================
--- llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (original)
+++ llvm/branches/Apple/Morbo/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp Thu Apr  8 17:30:50 2010
@@ -3822,8 +3822,6 @@
     return 0;
   }
   case Intrinsic::dbg_value: {
-    // FIXME: We are not yet ready to handle dbg_value.
-    return 0;
     DwarfWriter *DW = DAG.getDwarfWriter();
     if (!DW)
       return 0;

Modified: llvm/branches/Apple/Morbo/test/DebugInfo/2010-01-18-DbgValue.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/Apple/Morbo/test/DebugInfo/2010-01-18-DbgValue.ll?rev=100821&r1=100820&r2=100821&view=diff
==============================================================================
--- llvm/branches/Apple/Morbo/test/DebugInfo/2010-01-18-DbgValue.ll (original)
+++ llvm/branches/Apple/Morbo/test/DebugInfo/2010-01-18-DbgValue.ll Thu Apr  8 17:30:50 2010
@@ -4,7 +4,7 @@
 target triple = "i386-apple-darwin9.8"
 ; Currently, dbg.declare generates a DEBUG_VALUE comment.  Eventually it will
 ; generate DWARF and this test will need to be modified or removed.
-; XFAIL: *
+
 @Y = common global i32 0                          ; <i32*> [#uses=1]
 
 define i32 @test() nounwind {