[llvm-commits] CVS: llvm/include/llvm/CodeGen/MachineInstr.h MachineInstrBuilder.h

Chris Lattner lattner at cs.uiuc.edu
Wed May 3 17:44:43 PDT 2006



Changes in directory llvm/include/llvm/CodeGen:

MachineInstr.h updated: 1.168 -> 1.169
MachineInstrBuilder.h updated: 1.30 -> 1.31
---
Log message:

Remove some more unused stuff from MachineInstr that was leftover from V9.


---
Diffs of the changes:  (+3 -71)

 MachineInstr.h        |   49 +++----------------------------------------------
 MachineInstrBuilder.h |   25 -------------------------
 2 files changed, 3 insertions(+), 71 deletions(-)


Index: llvm/include/llvm/CodeGen/MachineInstr.h
diff -u llvm/include/llvm/CodeGen/MachineInstr.h:1.168 llvm/include/llvm/CodeGen/MachineInstr.h:1.169
--- llvm/include/llvm/CodeGen/MachineInstr.h:1.168	Sat Apr 22 13:53:45 2006
+++ llvm/include/llvm/CodeGen/MachineInstr.h	Wed May  3 19:44:25 2006
@@ -98,10 +98,8 @@
   enum MachineOperandType {
     MO_VirtualRegister,         // virtual register for *value
     MO_MachineRegister,         // pre-assigned machine register `regNum'
-    MO_CCRegister,
     MO_SignExtendedImmed,
     MO_UnextendedImmed,
-    MO_PCRelativeDisp,
     MO_MachineBasicBlock,       // MachineBasicBlock reference
     MO_FrameIndex,              // Abstract Stack Frame Index
     MO_ConstantPoolIndex,       // Address of indexed Constant in Constant Pool
@@ -237,7 +235,6 @@
   /// Accessors that tell you what kind of MachineOperand you're looking at.
   ///
   bool isMachineBasicBlock() const { return opType == MO_MachineBasicBlock; }
-  bool isPCRelativeDisp() const { return opType == MO_PCRelativeDisp; }
   bool isImmediate() const {
     return opType == MO_SignExtendedImmed || opType == MO_UnextendedImmed;
   }
@@ -251,16 +248,14 @@
   /// has one. This is deprecated and only used by the SPARC v9 backend.
   ///
   Value* getVRegValueOrNull() const {
-    return (opType == MO_VirtualRegister || opType == MO_CCRegister ||
-            isPCRelativeDisp()) ? contents.value : NULL;
+    return opType == MO_VirtualRegister ? contents.value : NULL;
   }
 
   /// MachineOperand accessors that only work on certain types of
   /// MachineOperand...
   ///
   Value* getVRegValue() const {
-    assert ((opType == MO_VirtualRegister || opType == MO_CCRegister
-             || isPCRelativeDisp()) && "Wrong MachineOperand accessor");
+    assert(opType == MO_VirtualRegister && "Wrong MachineOperand accessor");
     return contents.value;
   }
   int getMachineRegNum() const {
@@ -322,8 +317,7 @@
   ///
   bool hasAllocatedReg() const {
     return (extra.regNum >= 0 &&
-            (opType == MO_VirtualRegister || opType == MO_CCRegister ||
-             opType == MO_MachineRegister));
+            (opType == MO_VirtualRegister || opType == MO_MachineRegister));
   }
 
   /// getReg - Returns the register number. It is a runtime error to call this
@@ -362,25 +356,6 @@
 
   friend std::ostream& operator<<(std::ostream& os, const MachineOperand& mop);
 
-  /// markHi32, markLo32, etc. - These methods are deprecated and only used by
-  /// the SPARC v9 back-end.
-  ///
-  void markHi32()      { flags |= HIFLAG32; }
-  void markLo32()      { flags |= LOFLAG32; }
-  void markHi64()      { flags |= HIFLAG64; }
-  void markLo64()      { flags |= LOFLAG64; }
-
-private:
-  /// setRegForValue - Replaces the Value with its corresponding physical
-  /// register after register allocation is complete. This is deprecated
-  /// and only used by the SPARC v9 back-end.
-  ///
-  void setRegForValue(int reg) {
-    assert(opType == MO_VirtualRegister || opType == MO_CCRegister ||
-           opType == MO_MachineRegister);
-    extra.regNum = reg;
-  }
-
   friend class MachineInstr;
 };
 
@@ -507,15 +482,6 @@
                                       UTy, isPCRelative));
   }
 
-  void addCCRegOperand(Value *V,
-                       MachineOperand::UseType UTy = MachineOperand::Use) {
-    assert(!OperandsComplete() &&
-           "Trying to add an operand to a machine instr that is already done!");
-    operands.push_back(MachineOperand(V, MachineOperand::MO_CCRegister, UTy,
-                                      false));
-  }
-
-
   /// addRegOperand - Add a symbolic virtual register reference...
   ///
   void addRegOperand(int reg, bool isDef) {
@@ -536,15 +502,6 @@
       MachineOperand(reg, MachineOperand::MO_VirtualRegister, UTy));
   }
 
-  /// addPCDispOperand - Add a PC relative displacement operand to the MI
-  ///
-  void addPCDispOperand(Value *V) {
-    assert(!OperandsComplete() &&
-           "Trying to add an operand to a machine instr that is already done!");
-    operands.push_back(
-      MachineOperand(V, MachineOperand::MO_PCRelativeDisp,MachineOperand::Use));
-  }
-
   /// addMachineRegOperand - Add a virtual register operand to this MachineInstr
   ///
   void addMachineRegOperand(int reg, bool isDef) {


Index: llvm/include/llvm/CodeGen/MachineInstrBuilder.h
diff -u llvm/include/llvm/CodeGen/MachineInstrBuilder.h:1.30 llvm/include/llvm/CodeGen/MachineInstrBuilder.h:1.31
--- llvm/include/llvm/CodeGen/MachineInstrBuilder.h:1.30	Mon Apr 24 01:42:15 2006
+++ llvm/include/llvm/CodeGen/MachineInstrBuilder.h	Wed May  3 19:44:25 2006
@@ -55,15 +55,6 @@
     return *this;
   }
 
-  /// addReg - Add an LLVM value that is to be used as a register...
-  ///
-  const MachineInstrBuilder &addCCReg(
-    Value *V,
-    MachineOperand::UseType Ty = MachineOperand::Use) const {
-    MI->addCCRegOperand(V, Ty);
-    return *this;
-  }
-
   /// addRegDef - Add an LLVM value that is to be defined as a register... this
   /// is the same as addReg(V, MachineOperand::Def).
   ///
@@ -71,22 +62,6 @@
     return addReg(V, MachineOperand::Def);
   }
 
-  /// addPCDisp - Add an LLVM value to be treated as a PC relative
-  /// displacement...
-  ///
-  const MachineInstrBuilder &addPCDisp(Value *V) const {
-    MI->addPCDispOperand(V);
-    return *this;
-  }
-
-  /// addMReg - Add a machine register operand...
-  ///
-  const MachineInstrBuilder &addMReg(int Reg, MachineOperand::UseType Ty
-                                        = MachineOperand::Use) const {
-    MI->addMachineRegOperand(Reg, Ty);
-    return *this;
-  }
-
   /// addImm - Add a new immediate operand.
   ///
   const MachineInstrBuilder &addImm(int Val) const {






More information about the llvm-commits mailing list