[llvm-commits] [llvm] r58591 - /llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp

[Owen Anderson]

Author: resistor
Date: Sun Nov  2 20:33:28 2008
New Revision: 58591

URL: http://llvm.org/viewvc/llvm-project?rev=58591&view=rev
Log:
Revert my last patch until I consult with Evan about it.

Modified:
    llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp

Modified: llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp?rev=58591&r1=58590&r2=58591&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp (original)
+++ llvm/trunk/lib/CodeGen/PreAllocSplitting.cpp Sun Nov  2 20:33:28 2008
@@ -89,8 +89,6 @@
         AU.addPreservedID(StrongPHIEliminationID);
       else
         AU.addPreservedID(PHIEliminationID);
-      AU.addRequired<MachineLoopInfo>();
-      AU.addPreserved<MachineLoopInfo>();
       MachineFunctionPass::getAnalysisUsage(AU);
     }
     
@@ -635,14 +633,6 @@
     assert(0 && "Val# is defined by a dead def?");
     abort();
   }
-  
-  // Pre-splitting a vreg that does not have a PHI kill across a barrier
-  // that is within a loop can potentially create a join that was not
-  // present before, which would make updating the live intervals very
-  // difficult.  Bailout instead.
-  MachineLoopInfo& MLI = getAnalysis<MachineLoopInfo>();
-  if (!ValNo->hasPHIKill && MLI.getLoopFor(BarrierMBB))
-    return false;
 
   // FIXME: For now, if definition is rematerializable, do not split.
   MachineInstr *DefMI = (ValNo->def != ~0U)