Fix a bug where we didn't insert FP_REG_KILL instructions into MBB's that

contain FP PHI nodes but no other FP defining instructions.  This fixes
183.equake

llvm-svn: 19495

1 files changed, 15 insertions, 0 deletions

diff --git a/llvm/lib/Target/X86/X86ISelPattern.cpp b/llvm/lib/Target/X86/X86ISelPattern.cpp
index 34d13564707..22c926a855b 100644
--- a/llvm/lib/Target/X86/X86ISelPattern.cpp
+++ b/llvm/lib/Target/X86/X86ISelPattern.cpp
@@ -331,6 +331,21 @@ namespace {
       // FP_REG_KILL insertion.
       ContainsFPCode = false;
 
+      // Scan the PHI nodes that already are inserted into this basic block.  If
+      // any of them is a PHI of a floating point value, we need to insert an
+      // FP_REG_KILL.
+      SSARegMap *RegMap = BB->getParent()->getSSARegMap();
+      for (MachineBasicBlock::iterator I = BB->begin(), E = BB->end();
+           I != E; ++I) {
+        assert(I->getOpcode() == X86::PHI &&
+               "Isn't just PHI nodes?");
+        if (RegMap->getRegClass(I->getOperand(0).getReg()) ==
+            X86::RFPRegisterClass) {
+          ContainsFPCode = true;
+          break;
+        }
+      }
+
       // Compute the RegPressureMap, which is an approximation for the number of
       // registers required to compute each node.
       ComputeRegPressure(DAG.getRoot());