[llvm-commits] CVS: llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp

[Brian Gaeke]

Changes in directory llvm/lib/Target/SparcV8:

SparcV8InstrInfo.cpp updated: 1.6 -> 1.7
---
Log message:

Look for many more moves to fold (previously, we only
*or g0, x      add g0, x          recognized * as a move)
 or  x, g0     add  x, g0
 or  0, x      add  0, x
 or  x, 0      add  x, 0


---
Diffs of the changes:  (+24 -2)

Index: llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp
diff -u llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp:1.6 llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp:1.7
--- llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp:1.6	Wed Sep 29 11:45:47 2004
+++ llvm/lib/Target/SparcV8/SparcV8InstrInfo.cpp	Fri Dec 10 23:19:03 2004
@@ -21,16 +21,38 @@
   : TargetInstrInfo(SparcV8Insts, sizeof(SparcV8Insts)/sizeof(SparcV8Insts[0])){
 }
 
+static bool isZeroImmed (const MachineOperand &op) {
+  return (op.isImmediate() && op.getImmedValue() == 0);
+}
+
 /// Return true if the instruction is a register to register move and
 /// leave the source and dest operands in the passed parameters.
 ///
 bool SparcV8InstrInfo::isMoveInstr(const MachineInstr &MI,
                                    unsigned &SrcReg, unsigned &DstReg) const {
-  if (MI.getOpcode() == V8::ORrr) {
-    if (MI.getOperand(1).getReg() == V8::G0) {  // X = or G0, Y -> X = Y
+  // We look for 3 kinds of patterns here:
+  // or with G0 or 0
+  // add with G0 or 0
+  // fmovs or FpMOVD (pseudo double move).
+  if (MI.getOpcode() == V8::ORrr || MI.getOpcode() == V8::ADDrr) {
+    if (MI.getOperand(1).getReg() == V8::G0) {
       DstReg = MI.getOperand(0).getReg();
       SrcReg = MI.getOperand(2).getReg();
       return true;
+    } else if (MI.getOperand (2).getReg() == V8::G0) {
+      DstReg = MI.getOperand(0).getReg();
+      SrcReg = MI.getOperand(1).getReg();
+      return true;
+    }
+  } else if (MI.getOpcode() == V8::ORri || MI.getOpcode() == V8::ADDri) {
+    if (isZeroImmed (MI.getOperand (1))) {
+      DstReg = MI.getOperand(0).getReg();
+      SrcReg = MI.getOperand(2).getReg();
+      return true;
+    } else if (isZeroImmed (MI.getOperand (2))) {
+      DstReg = MI.getOperand(0).getReg();
+      SrcReg = MI.getOperand(1).getReg();
+      return true;
     }
   } else if (MI.getOpcode() == V8::FMOVS || MI.getOpcode() == V8::FpMOVD) {
     SrcReg = MI.getOperand(1).getReg();