[llvm-commits] CVS: llvm/lib/Target/X86/X86RegisterInfo.cpp

[Chris Lattner]

Changes in directory llvm/lib/Target/X86:

X86RegisterInfo.cpp updated: 1.56 -> 1.57

---
Log message:

Add a (hidden) option to print instructions that fail to fuse.  It's looking
like compares and test's would be the next huge win...


---
Diffs of the changes:  (+16 -8)

Index: llvm/lib/Target/X86/X86RegisterInfo.cpp
diff -u llvm/lib/Target/X86/X86RegisterInfo.cpp:1.56 llvm/lib/Target/X86/X86RegisterInfo.cpp:1.57
--- llvm/lib/Target/X86/X86RegisterInfo.cpp:1.56	Tue Feb 17 01:47:20 2004
+++ llvm/lib/Target/X86/X86RegisterInfo.cpp	Tue Feb 17 02:03:47 2004
@@ -34,6 +34,11 @@
   cl::opt<bool>
   NoFusing("disable-spill-fusing",
            cl::desc("Disable fusing of spill code into instructions"));
+  cl::opt<bool>
+  PrintFailedFusing("print-failed-fuse-candidates",
+                    cl::desc("Print instructions that the allocator wants to"
+                             " fuse, but the X86 backend currently can't"),
+                    cl::Hidden);
 }
 
 X86RegisterInfo::X86RegisterInfo()
@@ -135,7 +140,7 @@
     case X86::ANDri8:  NI = MakeMIInst(X86::ANDmi8 , FrameIndex, MI); break;
     case X86::ANDri16: NI = MakeMIInst(X86::ANDmi16, FrameIndex, MI); break;
     case X86::ANDri32: NI = MakeMIInst(X86::ANDmi32, FrameIndex, MI); break;
-    default: return false; // Cannot fold
+    default: break; // Cannot fold
     }
   } else if (i == 1) {
     switch(MI->getOpcode()) {
@@ -150,16 +155,19 @@
     case X86::ANDrr32: NI = MakeRMInst(X86::ANDrm32, FrameIndex, MI); break;
     case X86::IMULrr16:NI = MakeRMInst(X86::IMULrm16, FrameIndex, MI); break;
     case X86::IMULrr32:NI = MakeRMInst(X86::IMULrm32, FrameIndex, MI); break;
-    case X86::IMULrri16: NI = MakeRMIInst(X86::IMULrmi16, FrameIndex, MI); break;
-    case X86::IMULrri32: NI = MakeRMIInst(X86::IMULrmi32, FrameIndex, MI); break;
-    default: return false;  // cannot fold.
+    case X86::IMULrri16: NI = MakeRMIInst(X86::IMULrmi16, FrameIndex, MI);break;
+    case X86::IMULrri32: NI = MakeRMIInst(X86::IMULrmi32, FrameIndex, MI);break;
+    default: break;
     }
+  }
+  if (NI) {
+    MI = MBB.insert(MBB.erase(MI), NI);
+    return true;
   } else {
-    return false; // cannot fold.
+    if (PrintFailedFusing)
+      std::cerr << "We failed to fuse: " << *MI;
+    return false;
   }
-  
-  MI = MBB.insert(MBB.erase(MI), NI);
-  return true;
 }
 
 //===----------------------------------------------------------------------===//