Add a new SSA-based peephole optimizer which includes copy propagation and
authorChris Lattner <sabre@nondot.org>
Mon, 1 Dec 2003 05:15:28 +0000 (05:15 +0000)
committerChris Lattner <sabre@nondot.org>
Mon, 1 Dec 2003 05:15:28 +0000 (05:15 +0000)
folding of instructions into addressing modes.  This creates lots of dead
instructions, which are currently not deleted.  It also creates a lot of
instructions that the X86 backend currently cannot handle.  :(

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10275 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/PeepholeOptimizer.cpp
lib/Target/X86/X86PeepholeOpt.cpp

index 488dc8914594c8c6ce9f00c02dce8da5942b36d4..4c887e1cff67afe2d84befcd3e749e4a0cfd41ad 100644 (file)
 #include "X86.h"
 #include "llvm/CodeGen/MachineFunctionPass.h"
 #include "llvm/CodeGen/MachineInstrBuilder.h"
+#include "Support/Statistic.h"
 using namespace llvm;
 
 namespace {
+  Statistic<> NumPHOpts("x86-peephole",
+                        "Number of peephole optimization performed");
   struct PH : public MachineFunctionPass {
     virtual bool runOnMachineFunction(MachineFunction &MF);
 
@@ -34,9 +37,10 @@ bool PH::runOnMachineFunction(MachineFunction &MF) {
 
   for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI != E; ++BI)
     for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); )
-      if (PeepholeOptimize(*BI, I))
+      if (PeepholeOptimize(*BI, I)) {
        Changed = true;
-      else
+        ++NumPHOpts;
+      } else
        ++I;
 
   return Changed;
@@ -133,3 +137,280 @@ bool PH::PeepholeOptimize(MachineBasicBlock &MBB,
   }
 }
 
+namespace {
+  class UseDefChains : public MachineFunctionPass {
+    std::vector<MachineInstr*> DefiningInst;
+  public:
+    // getDefinition - Return the machine instruction that defines the specified
+    // SSA virtual register.
+    MachineInstr *getDefinition(unsigned Reg) {
+      assert(Reg >= MRegisterInfo::FirstVirtualRegister &&
+             "use-def chains only exist for SSA registers!");
+      assert(Reg - MRegisterInfo::FirstVirtualRegister < DefiningInst.size() &&
+             "Unknown register number!");
+      assert(DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] &&
+             "Unknown register number!");
+      return DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister];
+    }
+
+    // setDefinition - Update the use-def chains to indicate that MI defines
+    // register Reg.
+    void setDefinition(unsigned Reg, MachineInstr *MI) {
+      if (Reg-MRegisterInfo::FirstVirtualRegister >= DefiningInst.size())
+        DefiningInst.resize(Reg-MRegisterInfo::FirstVirtualRegister+1);
+      DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] = MI;
+    }
+
+    // removeDefinition - Update the use-def chains to forget about Reg
+    // entirely.
+    void removeDefinition(unsigned Reg) {
+      assert(getDefinition(Reg));      // Check validity
+      DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] = 0;
+    }
+
+    virtual bool runOnMachineFunction(MachineFunction &MF) {
+      for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI!=E; ++BI)
+        for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); ++I) {
+          MachineInstr *MI = *I;
+          for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
+            MachineOperand &MO = MI->getOperand(i);
+            if (MO.isVirtualRegister() && MO.opIsDefOnly())
+              setDefinition(MO.getReg(), MI);
+          }
+        }
+      return false;
+    }
+
+    virtual void getAnalysisUsage(AnalysisUsage &AU) const {
+      AU.setPreservesAll();
+      MachineFunctionPass::getAnalysisUsage(AU);
+    }
+
+    virtual void releaseMemory() {
+      std::vector<MachineInstr*>().swap(DefiningInst);
+    }
+  };
+
+  RegisterAnalysis<UseDefChains> X("use-def-chains",
+                                "use-def chain construction for machine code");
+}
+
+
+namespace {
+  Statistic<> NumSSAPHOpts("x86-ssa-peephole",
+                           "Number of SSA peephole optimization performed");
+
+  /// SSAPH - This pass is an X86-specific, SSA-based, peephole optimizer.  This
+  /// pass is really a bad idea: a better instruction selector should completely
+  /// supersume it.  However, that will take some time to develop, and the
+  /// simple things this can do are important now.
+  class SSAPH : public MachineFunctionPass {
+    UseDefChains *UDC;
+  public:
+    virtual bool runOnMachineFunction(MachineFunction &MF);
+
+    bool PeepholeOptimize(MachineBasicBlock &MBB,
+                         MachineBasicBlock::iterator &I);
+
+    virtual const char *getPassName() const {
+      return "X86 SSA-based Peephole Optimizer";
+    }
+
+    /// Propagate - Set MI[DestOpNo] = Src[SrcOpNo], optionally change the
+    /// opcode of the instruction, then return true.
+    bool Propagate(MachineInstr *MI, unsigned DestOpNo,
+                   MachineInstr *Src, unsigned SrcOpNo, unsigned NewOpcode = 0){
+      MI->getOperand(DestOpNo) = Src->getOperand(SrcOpNo);
+      if (NewOpcode) MI->setOpcode(NewOpcode);
+      return true;
+    }
+
+    /// OptimizeAddress - If we can fold the addressing arithmetic for this
+    /// memory instruction into the instruction itself, do so and return true.
+    bool OptimizeAddress(MachineInstr *MI, unsigned OpNo);
+
+    /// getDefininingInst - If the specified operand is a read of an SSA
+    /// register, return the machine instruction defining it, otherwise, return
+    /// null.
+    MachineInstr *getDefiningInst(MachineOperand &MO) {
+      if (!MO.opIsUse() || !MO.isVirtualRegister()) return 0;
+      return UDC->getDefinition(MO.getReg());
+    }
+
+    virtual void getAnalysisUsage(AnalysisUsage &AU) const {
+      AU.addRequired<UseDefChains>();
+      AU.addPreserved<UseDefChains>();
+      MachineFunctionPass::getAnalysisUsage(AU);
+    }
+  };
+}
+
+FunctionPass *llvm::createX86SSAPeepholeOptimizerPass() { return new SSAPH(); }
+
+bool SSAPH::runOnMachineFunction(MachineFunction &MF) {
+  bool Changed = false;
+  bool LocalChanged;
+
+  UDC = &getAnalysis<UseDefChains>();
+
+  do {
+    LocalChanged = false;
+
+    for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI != E; ++BI)
+      for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); )
+        if (PeepholeOptimize(*BI, I)) {
+          LocalChanged = true;
+          ++NumSSAPHOpts;
+        } else
+          ++I;
+    Changed |= LocalChanged;
+  } while (LocalChanged);
+
+  return Changed;
+}
+
+static bool isValidScaleAmount(unsigned Scale) {
+  return Scale == 1 || Scale == 2 || Scale == 4 || Scale == 8;
+}
+
+/// OptimizeAddress - If we can fold the addressing arithmetic for this
+/// memory instruction into the instruction itself, do so and return true.
+bool SSAPH::OptimizeAddress(MachineInstr *MI, unsigned OpNo) {
+  MachineOperand &BaseRegOp      = MI->getOperand(OpNo+0);
+  MachineOperand &ScaleOp        = MI->getOperand(OpNo+1);
+  MachineOperand &IndexRegOp     = MI->getOperand(OpNo+2);
+  MachineOperand &DisplacementOp = MI->getOperand(OpNo+3);
+
+  unsigned BaseReg  = BaseRegOp.hasAllocatedReg() ? BaseRegOp.getReg() : 0;
+  unsigned Scale    = ScaleOp.getImmedValue();
+  unsigned IndexReg = IndexRegOp.hasAllocatedReg() ? IndexRegOp.getReg() : 0;
+
+  bool Changed = false;
+
+  // If the base register is unset, and the index register is set with a scale
+  // of 1, move it to be the base register.
+  if (BaseRegOp.hasAllocatedReg() && BaseReg == 0 &&
+      Scale == 1 && IndexReg != 0) {
+    BaseRegOp.setReg(IndexReg);
+    IndexRegOp.setReg(0);
+    return true;
+  }
+
+  // Attempt to fold instructions used by the base register into the instruction
+  if (MachineInstr *DefInst = getDefiningInst(BaseRegOp)) {
+    switch (DefInst->getOpcode()) {
+    case X86::MOVir32:
+      // If there is no displacement set for this instruction set one now.
+      // FIXME: If we can fold two immediates together, we should do so!
+      if (DisplacementOp.isImmediate() && !DisplacementOp.getImmedValue()) {
+        if (DefInst->getOperand(1).isImmediate()) {
+          BaseRegOp.setReg(0);
+          return Propagate(MI, OpNo+3, DefInst, 1);
+        }
+      }
+      break;
+
+    case X86::ADDrr32:
+      // If the source is a register-register add, and we do not yet have an
+      // index register, fold the add into the memory address.
+      if (IndexReg == 0) {
+        BaseRegOp = DefInst->getOperand(1);
+        IndexRegOp = DefInst->getOperand(2);
+        ScaleOp.setImmedValue(1);
+        return true;
+      }
+      break;
+
+    case X86::SHLir32:
+      // If this shift could be folded into the index portion of the address if
+      // it were the index register, move it to the index register operand now,
+      // so it will be folded in below.
+      if ((Scale == 1 || (IndexReg == 0 && IndexRegOp.hasAllocatedReg())) &&
+          DefInst->getOperand(2).getImmedValue() < 4) {
+        std::swap(BaseRegOp, IndexRegOp);
+        ScaleOp.setImmedValue(1); Scale = 1;
+        std::swap(IndexReg, BaseReg);
+        Changed = true;
+        break;
+      }
+    }
+  }
+
+  // Attempt to fold instructions used by the index into the instruction
+  if (MachineInstr *DefInst = getDefiningInst(IndexRegOp)) {
+    switch (DefInst->getOpcode()) {
+    case X86::SHLir32: {
+      // Figure out what the resulting scale would be if we folded this shift.
+      unsigned ResScale = Scale * (1 << DefInst->getOperand(2).getImmedValue());
+      if (isValidScaleAmount(ResScale)) {
+        IndexRegOp = DefInst->getOperand(1);
+        ScaleOp.setImmedValue(ResScale);
+        return true;
+      }
+      break;
+    }
+    }
+  }
+
+  return Changed;
+}
+
+bool SSAPH::PeepholeOptimize(MachineBasicBlock &MBB,
+                             MachineBasicBlock::iterator &I) {
+  MachineInstr *MI = *I;
+  MachineInstr *Next = (I+1 != MBB.end()) ? *(I+1) : 0;
+
+  bool Changed = false;
+
+  // Scan the operands of this instruction.  If any operands are
+  // register-register copies, replace the operand with the source.
+  for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i)
+    // Is this an SSA register use?
+    if (MachineInstr *DefInst = getDefiningInst(MI->getOperand(i)))
+      // If the operand is a vreg-vreg copy, it is always safe to replace the
+      // source value with the input operand.
+      if (DefInst->getOpcode() == X86::MOVrr8  ||
+          DefInst->getOpcode() == X86::MOVrr16 ||
+          DefInst->getOpcode() == X86::MOVrr32) {
+        // Don't propagate physical registers into PHI nodes...
+        if (MI->getOpcode() != X86::PHI ||
+            DefInst->getOperand(1).isVirtualRegister())
+        Changed = Propagate(MI, i, DefInst, 1);
+      }
+  
+  
+  // Perform instruction specific optimizations.
+  switch (MI->getOpcode()) {
+
+    // Register to memory stores.  Format: <base,scale,indexreg,immdisp>, srcreg
+  case X86::MOVrm32: case X86::MOVrm16: case X86::MOVrm8:
+  case X86::MOVim32: case X86::MOVim16: case X86::MOVim8:
+    // Check to see if we can fold the source instruction into this one...
+    if (MachineInstr *SrcInst = getDefiningInst(MI->getOperand(4))) {
+      switch (SrcInst->getOpcode()) {
+        // Fold the immediate value into the store, if possible.
+      case X86::MOVir8:  return Propagate(MI, 4, SrcInst, 1, X86::MOVim8);
+      case X86::MOVir16: return Propagate(MI, 4, SrcInst, 1, X86::MOVim16);
+      case X86::MOVir32: return Propagate(MI, 4, SrcInst, 1, X86::MOVim32);
+      default: break;
+      }
+    }
+
+    // If we can optimize the addressing expression, do so now.
+    if (OptimizeAddress(MI, 0))
+      return true;
+    break;
+
+  case X86::MOVmr32:
+  case X86::MOVmr16:
+  case X86::MOVmr8:
+    // If we can optimize the addressing expression, do so now.
+    if (OptimizeAddress(MI, 1))
+      return true;
+    break;
+
+  default: break;
+  }
+
+  return Changed;
+}
index 488dc8914594c8c6ce9f00c02dce8da5942b36d4..4c887e1cff67afe2d84befcd3e749e4a0cfd41ad 100644 (file)
 #include "X86.h"
 #include "llvm/CodeGen/MachineFunctionPass.h"
 #include "llvm/CodeGen/MachineInstrBuilder.h"
+#include "Support/Statistic.h"
 using namespace llvm;
 
 namespace {
+  Statistic<> NumPHOpts("x86-peephole",
+                        "Number of peephole optimization performed");
   struct PH : public MachineFunctionPass {
     virtual bool runOnMachineFunction(MachineFunction &MF);
 
@@ -34,9 +37,10 @@ bool PH::runOnMachineFunction(MachineFunction &MF) {
 
   for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI != E; ++BI)
     for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); )
-      if (PeepholeOptimize(*BI, I))
+      if (PeepholeOptimize(*BI, I)) {
        Changed = true;
-      else
+        ++NumPHOpts;
+      } else
        ++I;
 
   return Changed;
@@ -133,3 +137,280 @@ bool PH::PeepholeOptimize(MachineBasicBlock &MBB,
   }
 }
 
+namespace {
+  class UseDefChains : public MachineFunctionPass {
+    std::vector<MachineInstr*> DefiningInst;
+  public:
+    // getDefinition - Return the machine instruction that defines the specified
+    // SSA virtual register.
+    MachineInstr *getDefinition(unsigned Reg) {
+      assert(Reg >= MRegisterInfo::FirstVirtualRegister &&
+             "use-def chains only exist for SSA registers!");
+      assert(Reg - MRegisterInfo::FirstVirtualRegister < DefiningInst.size() &&
+             "Unknown register number!");
+      assert(DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] &&
+             "Unknown register number!");
+      return DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister];
+    }
+
+    // setDefinition - Update the use-def chains to indicate that MI defines
+    // register Reg.
+    void setDefinition(unsigned Reg, MachineInstr *MI) {
+      if (Reg-MRegisterInfo::FirstVirtualRegister >= DefiningInst.size())
+        DefiningInst.resize(Reg-MRegisterInfo::FirstVirtualRegister+1);
+      DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] = MI;
+    }
+
+    // removeDefinition - Update the use-def chains to forget about Reg
+    // entirely.
+    void removeDefinition(unsigned Reg) {
+      assert(getDefinition(Reg));      // Check validity
+      DefiningInst[Reg-MRegisterInfo::FirstVirtualRegister] = 0;
+    }
+
+    virtual bool runOnMachineFunction(MachineFunction &MF) {
+      for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI!=E; ++BI)
+        for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); ++I) {
+          MachineInstr *MI = *I;
+          for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
+            MachineOperand &MO = MI->getOperand(i);
+            if (MO.isVirtualRegister() && MO.opIsDefOnly())
+              setDefinition(MO.getReg(), MI);
+          }
+        }
+      return false;
+    }
+
+    virtual void getAnalysisUsage(AnalysisUsage &AU) const {
+      AU.setPreservesAll();
+      MachineFunctionPass::getAnalysisUsage(AU);
+    }
+
+    virtual void releaseMemory() {
+      std::vector<MachineInstr*>().swap(DefiningInst);
+    }
+  };
+
+  RegisterAnalysis<UseDefChains> X("use-def-chains",
+                                "use-def chain construction for machine code");
+}
+
+
+namespace {
+  Statistic<> NumSSAPHOpts("x86-ssa-peephole",
+                           "Number of SSA peephole optimization performed");
+
+  /// SSAPH - This pass is an X86-specific, SSA-based, peephole optimizer.  This
+  /// pass is really a bad idea: a better instruction selector should completely
+  /// supersume it.  However, that will take some time to develop, and the
+  /// simple things this can do are important now.
+  class SSAPH : public MachineFunctionPass {
+    UseDefChains *UDC;
+  public:
+    virtual bool runOnMachineFunction(MachineFunction &MF);
+
+    bool PeepholeOptimize(MachineBasicBlock &MBB,
+                         MachineBasicBlock::iterator &I);
+
+    virtual const char *getPassName() const {
+      return "X86 SSA-based Peephole Optimizer";
+    }
+
+    /// Propagate - Set MI[DestOpNo] = Src[SrcOpNo], optionally change the
+    /// opcode of the instruction, then return true.
+    bool Propagate(MachineInstr *MI, unsigned DestOpNo,
+                   MachineInstr *Src, unsigned SrcOpNo, unsigned NewOpcode = 0){
+      MI->getOperand(DestOpNo) = Src->getOperand(SrcOpNo);
+      if (NewOpcode) MI->setOpcode(NewOpcode);
+      return true;
+    }
+
+    /// OptimizeAddress - If we can fold the addressing arithmetic for this
+    /// memory instruction into the instruction itself, do so and return true.
+    bool OptimizeAddress(MachineInstr *MI, unsigned OpNo);
+
+    /// getDefininingInst - If the specified operand is a read of an SSA
+    /// register, return the machine instruction defining it, otherwise, return
+    /// null.
+    MachineInstr *getDefiningInst(MachineOperand &MO) {
+      if (!MO.opIsUse() || !MO.isVirtualRegister()) return 0;
+      return UDC->getDefinition(MO.getReg());
+    }
+
+    virtual void getAnalysisUsage(AnalysisUsage &AU) const {
+      AU.addRequired<UseDefChains>();
+      AU.addPreserved<UseDefChains>();
+      MachineFunctionPass::getAnalysisUsage(AU);
+    }
+  };
+}
+
+FunctionPass *llvm::createX86SSAPeepholeOptimizerPass() { return new SSAPH(); }
+
+bool SSAPH::runOnMachineFunction(MachineFunction &MF) {
+  bool Changed = false;
+  bool LocalChanged;
+
+  UDC = &getAnalysis<UseDefChains>();
+
+  do {
+    LocalChanged = false;
+
+    for (MachineFunction::iterator BI = MF.begin(), E = MF.end(); BI != E; ++BI)
+      for (MachineBasicBlock::iterator I = BI->begin(); I != BI->end(); )
+        if (PeepholeOptimize(*BI, I)) {
+          LocalChanged = true;
+          ++NumSSAPHOpts;
+        } else
+          ++I;
+    Changed |= LocalChanged;
+  } while (LocalChanged);
+
+  return Changed;
+}
+
+static bool isValidScaleAmount(unsigned Scale) {
+  return Scale == 1 || Scale == 2 || Scale == 4 || Scale == 8;
+}
+
+/// OptimizeAddress - If we can fold the addressing arithmetic for this
+/// memory instruction into the instruction itself, do so and return true.
+bool SSAPH::OptimizeAddress(MachineInstr *MI, unsigned OpNo) {
+  MachineOperand &BaseRegOp      = MI->getOperand(OpNo+0);
+  MachineOperand &ScaleOp        = MI->getOperand(OpNo+1);
+  MachineOperand &IndexRegOp     = MI->getOperand(OpNo+2);
+  MachineOperand &DisplacementOp = MI->getOperand(OpNo+3);
+
+  unsigned BaseReg  = BaseRegOp.hasAllocatedReg() ? BaseRegOp.getReg() : 0;
+  unsigned Scale    = ScaleOp.getImmedValue();
+  unsigned IndexReg = IndexRegOp.hasAllocatedReg() ? IndexRegOp.getReg() : 0;
+
+  bool Changed = false;
+
+  // If the base register is unset, and the index register is set with a scale
+  // of 1, move it to be the base register.
+  if (BaseRegOp.hasAllocatedReg() && BaseReg == 0 &&
+      Scale == 1 && IndexReg != 0) {
+    BaseRegOp.setReg(IndexReg);
+    IndexRegOp.setReg(0);
+    return true;
+  }
+
+  // Attempt to fold instructions used by the base register into the instruction
+  if (MachineInstr *DefInst = getDefiningInst(BaseRegOp)) {
+    switch (DefInst->getOpcode()) {
+    case X86::MOVir32:
+      // If there is no displacement set for this instruction set one now.
+      // FIXME: If we can fold two immediates together, we should do so!
+      if (DisplacementOp.isImmediate() && !DisplacementOp.getImmedValue()) {
+        if (DefInst->getOperand(1).isImmediate()) {
+          BaseRegOp.setReg(0);
+          return Propagate(MI, OpNo+3, DefInst, 1);
+        }
+      }
+      break;
+
+    case X86::ADDrr32:
+      // If the source is a register-register add, and we do not yet have an
+      // index register, fold the add into the memory address.
+      if (IndexReg == 0) {
+        BaseRegOp = DefInst->getOperand(1);
+        IndexRegOp = DefInst->getOperand(2);
+        ScaleOp.setImmedValue(1);
+        return true;
+      }
+      break;
+
+    case X86::SHLir32:
+      // If this shift could be folded into the index portion of the address if
+      // it were the index register, move it to the index register operand now,
+      // so it will be folded in below.
+      if ((Scale == 1 || (IndexReg == 0 && IndexRegOp.hasAllocatedReg())) &&
+          DefInst->getOperand(2).getImmedValue() < 4) {
+        std::swap(BaseRegOp, IndexRegOp);
+        ScaleOp.setImmedValue(1); Scale = 1;
+        std::swap(IndexReg, BaseReg);
+        Changed = true;
+        break;
+      }
+    }
+  }
+
+  // Attempt to fold instructions used by the index into the instruction
+  if (MachineInstr *DefInst = getDefiningInst(IndexRegOp)) {
+    switch (DefInst->getOpcode()) {
+    case X86::SHLir32: {
+      // Figure out what the resulting scale would be if we folded this shift.
+      unsigned ResScale = Scale * (1 << DefInst->getOperand(2).getImmedValue());
+      if (isValidScaleAmount(ResScale)) {
+        IndexRegOp = DefInst->getOperand(1);
+        ScaleOp.setImmedValue(ResScale);
+        return true;
+      }
+      break;
+    }
+    }
+  }
+
+  return Changed;
+}
+
+bool SSAPH::PeepholeOptimize(MachineBasicBlock &MBB,
+                             MachineBasicBlock::iterator &I) {
+  MachineInstr *MI = *I;
+  MachineInstr *Next = (I+1 != MBB.end()) ? *(I+1) : 0;
+
+  bool Changed = false;
+
+  // Scan the operands of this instruction.  If any operands are
+  // register-register copies, replace the operand with the source.
+  for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i)
+    // Is this an SSA register use?
+    if (MachineInstr *DefInst = getDefiningInst(MI->getOperand(i)))
+      // If the operand is a vreg-vreg copy, it is always safe to replace the
+      // source value with the input operand.
+      if (DefInst->getOpcode() == X86::MOVrr8  ||
+          DefInst->getOpcode() == X86::MOVrr16 ||
+          DefInst->getOpcode() == X86::MOVrr32) {
+        // Don't propagate physical registers into PHI nodes...
+        if (MI->getOpcode() != X86::PHI ||
+            DefInst->getOperand(1).isVirtualRegister())
+        Changed = Propagate(MI, i, DefInst, 1);
+      }
+  
+  
+  // Perform instruction specific optimizations.
+  switch (MI->getOpcode()) {
+
+    // Register to memory stores.  Format: <base,scale,indexreg,immdisp>, srcreg
+  case X86::MOVrm32: case X86::MOVrm16: case X86::MOVrm8:
+  case X86::MOVim32: case X86::MOVim16: case X86::MOVim8:
+    // Check to see if we can fold the source instruction into this one...
+    if (MachineInstr *SrcInst = getDefiningInst(MI->getOperand(4))) {
+      switch (SrcInst->getOpcode()) {
+        // Fold the immediate value into the store, if possible.
+      case X86::MOVir8:  return Propagate(MI, 4, SrcInst, 1, X86::MOVim8);
+      case X86::MOVir16: return Propagate(MI, 4, SrcInst, 1, X86::MOVim16);
+      case X86::MOVir32: return Propagate(MI, 4, SrcInst, 1, X86::MOVim32);
+      default: break;
+      }
+    }
+
+    // If we can optimize the addressing expression, do so now.
+    if (OptimizeAddress(MI, 0))
+      return true;
+    break;
+
+  case X86::MOVmr32:
+  case X86::MOVmr16:
+  case X86::MOVmr8:
+    // If we can optimize the addressing expression, do so now.
+    if (OptimizeAddress(MI, 1))
+      return true;
+    break;
+
+  default: break;
+  }
+
+  return Changed;
+}