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

Chris Lattner lattner at cs.uiuc.edu
Thu Feb 2 23:06:36 PST 2006



Changes in directory llvm/lib/Target/SparcV8:

SparcV8RegisterInfo.cpp updated: 1.33 -> 1.34
SparcV8RegisterInfo.h updated: 1.6 -> 1.7
---
Log message:

Teach sparc to fold loads/stores into copies.
Remove the dead getRegClassForType method
minor formating changes.


---
Diffs of the changes:  (+47 -37)

 SparcV8RegisterInfo.cpp |   79 ++++++++++++++++++++++++++----------------------
 SparcV8RegisterInfo.h   |    5 ++-
 2 files changed, 47 insertions(+), 37 deletions(-)


Index: llvm/lib/Target/SparcV8/SparcV8RegisterInfo.cpp
diff -u llvm/lib/Target/SparcV8/SparcV8RegisterInfo.cpp:1.33 llvm/lib/Target/SparcV8/SparcV8RegisterInfo.cpp:1.34
--- llvm/lib/Target/SparcV8/SparcV8RegisterInfo.cpp:1.33	Mon Jan  9 12:28:21 2006
+++ llvm/lib/Target/SparcV8/SparcV8RegisterInfo.cpp	Fri Feb  3 01:06:25 2006
@@ -27,34 +27,29 @@
 
 void SparcV8RegisterInfo::
 storeRegToStackSlot(MachineBasicBlock &MBB, MachineBasicBlock::iterator I,
-                    unsigned SrcReg, int FrameIdx,
+                    unsigned SrcReg, int FI,
                     const TargetRegisterClass *RC) const {
   // On the order of operands here: think "[FrameIdx + 0] = SrcReg".
   if (RC == V8::IntRegsRegisterClass)
-    BuildMI (MBB, I, V8::STri, 3).addFrameIndex (FrameIdx).addSImm (0)
-      .addReg (SrcReg);
+    BuildMI(MBB, I, V8::STri, 3).addFrameIndex(FI).addImm(0).addReg(SrcReg);
   else if (RC == V8::FPRegsRegisterClass)
-    BuildMI (MBB, I, V8::STFri, 3).addFrameIndex (FrameIdx).addSImm (0)
-      .addReg (SrcReg);
+    BuildMI(MBB, I, V8::STFri, 3).addFrameIndex(FI).addImm(0).addReg(SrcReg);
   else if (RC == V8::DFPRegsRegisterClass)
-    BuildMI (MBB, I, V8::STDFri, 3).addFrameIndex (FrameIdx).addSImm (0)
-      .addReg (SrcReg);
+    BuildMI(MBB, I, V8::STDFri, 3).addFrameIndex(FI).addImm(0).addReg(SrcReg);
   else
-    assert (0 && "Can't store this register to stack slot");
+    assert(0 && "Can't store this register to stack slot");
 }
 
 void SparcV8RegisterInfo::
 loadRegFromStackSlot(MachineBasicBlock &MBB, MachineBasicBlock::iterator I,
-                     unsigned DestReg, int FrameIdx,
+                     unsigned DestReg, int FI,
                      const TargetRegisterClass *RC) const {
   if (RC == V8::IntRegsRegisterClass)
-    BuildMI (MBB, I, V8::LDri, 2, DestReg).addFrameIndex (FrameIdx).addSImm (0);
+    BuildMI(MBB, I, V8::LDri, 2, DestReg).addFrameIndex(FI).addImm(0);
   else if (RC == V8::FPRegsRegisterClass)
-    BuildMI (MBB, I, V8::LDFri, 2, DestReg).addFrameIndex (FrameIdx)
-      .addSImm (0);
+    BuildMI(MBB, I, V8::LDFri, 2, DestReg).addFrameIndex(FI).addImm (0);
   else if (RC == V8::DFPRegsRegisterClass)
-    BuildMI (MBB, I, V8::LDDFri, 2, DestReg).addFrameIndex (FrameIdx)
-      .addSImm (0);
+    BuildMI(MBB, I, V8::LDDFri, 2, DestReg).addFrameIndex(FI).addImm(0);
   else
     assert(0 && "Can't load this register from stack slot");
 }
@@ -64,15 +59,46 @@
                                        unsigned DestReg, unsigned SrcReg,
                                        const TargetRegisterClass *RC) const {
   if (RC == V8::IntRegsRegisterClass)
-    BuildMI (MBB, I, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (SrcReg);
+    BuildMI(MBB, I, V8::ORrr, 2, DestReg).addReg(V8::G0).addReg(SrcReg);
   else if (RC == V8::FPRegsRegisterClass)
-    BuildMI (MBB, I, V8::FMOVS, 1, DestReg).addReg (SrcReg);
+    BuildMI(MBB, I, V8::FMOVS, 1, DestReg).addReg(SrcReg);
   else if (RC == V8::DFPRegsRegisterClass)
-    BuildMI (MBB, I, V8::FpMOVD, 1, DestReg).addReg (SrcReg);
+    BuildMI(MBB, I, V8::FpMOVD, 1, DestReg).addReg(SrcReg);
   else
     assert (0 && "Can't copy this register");
 }
 
+MachineInstr *SparcV8RegisterInfo::foldMemoryOperand(MachineInstr* MI,
+                                                     unsigned OpNum,
+                                                     int FI) const {
+  bool isFloat = false;
+  switch (MI->getOpcode()) {
+  case V8::ORrr:
+    if (MI->getOperand(1).isRegister() && MI->getOperand(1).getReg() == V8::G0&&
+        MI->getOperand(0).isRegister() && MI->getOperand(2).isRegister()) {
+      if (OpNum == 0)    // COPY -> STORE
+        return BuildMI(V8::STri, 3).addFrameIndex(FI).addImm(0)
+                                   .addReg(MI->getOperand(2).getReg());
+      else               // COPY -> LOAD
+        return BuildMI(V8::LDri, 2, MI->getOperand(0).getReg())
+                      .addFrameIndex(FI).addImm(0);
+    }
+    break;
+  case V8::FMOVS:
+    isFloat = true;
+    // FALLTHROUGH
+  case V8::FMOVD:
+    if (OpNum == 0)  // COPY -> STORE
+      return BuildMI(isFloat ? V8::STFri : V8::STDFri, 3)
+               .addFrameIndex(FI).addImm(0).addReg(MI->getOperand(1).getReg());
+    else             // COPY -> LOAD
+      return BuildMI(isFloat ? V8::LDFri : V8::LDDFri, 2, 
+                     MI->getOperand(0).getReg()).addFrameIndex(FI).addImm(0);
+    break;
+  }
+  return 0;
+}
+
 void SparcV8RegisterInfo::
 eliminateCallFramePseudoInstr(MachineFunction &MF, MachineBasicBlock &MBB,
                               MachineBasicBlock::iterator I) const {
@@ -172,22 +198,3 @@
 
 #include "SparcV8GenRegisterInfo.inc"
 
-const TargetRegisterClass*
-SparcV8RegisterInfo::getRegClassForType(const Type* Ty) const {
-  switch (Ty->getTypeID()) {
-  case Type::FloatTyID:  return V8::FPRegsRegisterClass;
-  case Type::DoubleTyID: return V8::DFPRegsRegisterClass;
-  case Type::LongTyID:
-  case Type::ULongTyID:  assert(0 && "Long values do not fit in registers!");
-  default:               assert(0 && "Invalid type to getClass!");
-  case Type::BoolTyID:
-  case Type::SByteTyID:
-  case Type::UByteTyID:
-  case Type::ShortTyID:
-  case Type::UShortTyID:
-  case Type::IntTyID:
-  case Type::UIntTyID:
-  case Type::PointerTyID: return V8::IntRegsRegisterClass;
-  }
-}
-


Index: llvm/lib/Target/SparcV8/SparcV8RegisterInfo.h
diff -u llvm/lib/Target/SparcV8/SparcV8RegisterInfo.h:1.6 llvm/lib/Target/SparcV8/SparcV8RegisterInfo.h:1.7
--- llvm/lib/Target/SparcV8/SparcV8RegisterInfo.h:1.6	Thu Sep 29 20:29:42 2005
+++ llvm/lib/Target/SparcV8/SparcV8RegisterInfo.h	Fri Feb  3 01:06:25 2006
@@ -23,7 +23,6 @@
 
 struct SparcV8RegisterInfo : public SparcV8GenRegisterInfo {
   SparcV8RegisterInfo();
-  const TargetRegisterClass* getRegClassForType(const Type* Ty) const;
 
   /// Code Generation virtual methods...
   void storeRegToStackSlot(MachineBasicBlock &MBB,
@@ -39,6 +38,10 @@
   void copyRegToReg(MachineBasicBlock &MBB, MachineBasicBlock::iterator MBBI,
                     unsigned DestReg, unsigned SrcReg,
                     const TargetRegisterClass *RC) const;
+  
+  virtual MachineInstr* foldMemoryOperand(MachineInstr* MI,
+                                          unsigned OpNum,
+                                          int FrameIndex) const;
 
   void eliminateCallFramePseudoInstr(MachineFunction &MF,
                                      MachineBasicBlock &MBB,






More information about the llvm-commits mailing list