Fix PR5024. LiveVariables physical register defs should *commit* only after all

[oota-llvm.git] / lib / CodeGen / LiveIntervalAnalysis.cpp
diff --git a/lib/CodeGen/LiveIntervalAnalysis.cpp b/lib/CodeGen/LiveIntervalAnalysis.cpp

index cb08fe759f509690e6c5e2ab84c7af43ee69c042..825cf396ab127aff026059ad30744bd1153f1bb0 100644 (file)
--- a/lib/CodeGen/LiveIntervalAnalysis.cpp
+++ b/lib/CodeGen/LiveIntervalAnalysis.cpp
@@ -23,6 +23,7 @@
  #include "llvm/CodeGen/LiveVariables.h"
  #include "llvm/CodeGen/MachineFrameInfo.h"
  #include "llvm/CodeGen/MachineInstr.h"
  #include "llvm/CodeGen/LiveVariables.h"
  #include "llvm/CodeGen/MachineFrameInfo.h"
  #include "llvm/CodeGen/MachineInstr.h"
+#include "llvm/CodeGen/MachineInstrBuilder.h"
  #include "llvm/CodeGen/MachineLoopInfo.h"
  #include "llvm/CodeGen/MachineRegisterInfo.h"
  #include "llvm/CodeGen/Passes.h"
  #include "llvm/CodeGen/MachineLoopInfo.h"
  #include "llvm/CodeGen/MachineRegisterInfo.h"
  #include "llvm/CodeGen/Passes.h"
@@ -33,9 +34,14 @@
  #include "llvm/Target/TargetOptions.h"
  #include "llvm/Support/CommandLine.h"
  #include "llvm/Support/Debug.h"
  #include "llvm/Target/TargetOptions.h"
  #include "llvm/Support/CommandLine.h"
  #include "llvm/Support/Debug.h"
+#include "llvm/Support/ErrorHandling.h"
+#include "llvm/Support/raw_ostream.h"
+#include "llvm/ADT/DepthFirstIterator.h"
+#include "llvm/ADT/SmallSet.h"
  #include "llvm/ADT/Statistic.h"
  #include "llvm/ADT/STLExtras.h"
  #include <algorithm>
  #include "llvm/ADT/Statistic.h"
  #include "llvm/ADT/STLExtras.h"
  #include <algorithm>
+#include <limits>
  #include <cmath>
  using namespace llvm;
  
  #include <cmath>
  using namespace llvm;
  
@@ -43,24 +49,26 @@ using namespace llvm;
  static cl::opt<bool> DisableReMat("disable-rematerialization", 
                                    cl::init(false), cl::Hidden);
  
  static cl::opt<bool> DisableReMat("disable-rematerialization", 
                                    cl::init(false), cl::Hidden);
  
-static cl::opt<bool> SplitAtBB("split-intervals-at-bb", 
-                               cl::init(true), cl::Hidden);
-static cl::opt<int> SplitLimit("split-limit",
-                               cl::init(-1), cl::Hidden);
-
  static cl::opt<bool> EnableAggressiveRemat("aggressive-remat", cl::Hidden);
  
  static cl::opt<bool> EnableFastSpilling("fast-spill",
                                          cl::init(false), cl::Hidden);
  
  static cl::opt<bool> EnableAggressiveRemat("aggressive-remat", cl::Hidden);
  
  static cl::opt<bool> EnableFastSpilling("fast-spill",
                                          cl::init(false), cl::Hidden);
  
-STATISTIC(numIntervals, "Number of original intervals");
-STATISTIC(numFolds    , "Number of loads/stores folded into instructions");
-STATISTIC(numSplits   , "Number of intervals split");
+static cl::opt<bool> EarlyCoalescing("early-coalescing", cl::init(false));
+
+static cl::opt<int> CoalescingLimit("early-coalescing-limit",
+                                    cl::init(-1), cl::Hidden);
+
+STATISTIC(numIntervals , "Number of original intervals");
+STATISTIC(numFolds     , "Number of loads/stores folded into instructions");
+STATISTIC(numSplits    , "Number of intervals split");
+STATISTIC(numCoalescing, "Number of early coalescing performed");
  
  char LiveIntervals::ID = 0;
  static RegisterPass<LiveIntervals> X("liveintervals", "Live Interval Analysis");
  
  void LiveIntervals::getAnalysisUsage(AnalysisUsage &AU) const {
  
  char LiveIntervals::ID = 0;
  static RegisterPass<LiveIntervals> X("liveintervals", "Live Interval Analysis");
  
  void LiveIntervals::getAnalysisUsage(AnalysisUsage &AU) const {
+  AU.setPreservesCFG();
    AU.addRequired<AliasAnalysis>();
    AU.addPreserved<AliasAnalysis>();
    AU.addPreserved<LiveVariables>();
    AU.addRequired<AliasAnalysis>();
    AU.addPreserved<AliasAnalysis>();
    AU.addPreserved<LiveVariables>();
@@ -88,15 +96,185 @@ void LiveIntervals::releaseMemory() {
    mi2iMap_.clear();
    i2miMap_.clear();
    r2iMap_.clear();
    mi2iMap_.clear();
    i2miMap_.clear();
    r2iMap_.clear();
+  terminatorGaps.clear();
+  phiJoinCopies.clear();
+
    // Release VNInfo memroy regions after all VNInfo objects are dtor'd.
    VNInfoAllocator.Reset();
    // Release VNInfo memroy regions after all VNInfo objects are dtor'd.
    VNInfoAllocator.Reset();
-  while (!ClonedMIs.empty()) {
-    MachineInstr *MI = ClonedMIs.back();
-    ClonedMIs.pop_back();
+  while (!CloneMIs.empty()) {
+    MachineInstr *MI = CloneMIs.back();
+    CloneMIs.pop_back();
      mf_->DeleteMachineInstr(MI);
    }
  }
  
      mf_->DeleteMachineInstr(MI);
    }
  }
  
+static bool CanTurnIntoImplicitDef(MachineInstr *MI, unsigned Reg,
+                                   unsigned OpIdx, const TargetInstrInfo *tii_){
+  unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
+  if (tii_->isMoveInstr(*MI, SrcReg, DstReg, SrcSubReg, DstSubReg) &&
+      Reg == SrcReg)
+    return true;
+
+  if (OpIdx == 2 && MI->getOpcode() == TargetInstrInfo::SUBREG_TO_REG)
+    return true;
+  if (OpIdx == 1 && MI->getOpcode() == TargetInstrInfo::EXTRACT_SUBREG)
+    return true;
+  return false;
+}
+
+/// processImplicitDefs - Process IMPLICIT_DEF instructions and make sure
+/// there is one implicit_def for each use. Add isUndef marker to
+/// implicit_def defs and their uses.
+void LiveIntervals::processImplicitDefs() {
+  SmallSet<unsigned, 8> ImpDefRegs;
+  SmallVector<MachineInstr*, 8> ImpDefMIs;
+  MachineBasicBlock *Entry = mf_->begin();
+  SmallPtrSet<MachineBasicBlock*,16> Visited;
+  for (df_ext_iterator<MachineBasicBlock*, SmallPtrSet<MachineBasicBlock*,16> >
+         DFI = df_ext_begin(Entry, Visited), E = df_ext_end(Entry, Visited);
+       DFI != E; ++DFI) {
+    MachineBasicBlock *MBB = *DFI;
+    for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
+         I != E; ) {
+      MachineInstr *MI = &*I;
+      ++I;
+      if (MI->getOpcode() == TargetInstrInfo::IMPLICIT_DEF) {
+        unsigned Reg = MI->getOperand(0).getReg();
+        ImpDefRegs.insert(Reg);
+        if (TargetRegisterInfo::isPhysicalRegister(Reg)) {
+          for (const unsigned *SS = tri_->getSubRegisters(Reg); *SS; ++SS)
+            ImpDefRegs.insert(*SS);
+        }
+        ImpDefMIs.push_back(MI);
+        continue;
+      }
+
+      if (MI->getOpcode() == TargetInstrInfo::INSERT_SUBREG) {
+        MachineOperand &MO = MI->getOperand(2);
+        if (ImpDefRegs.count(MO.getReg())) {
+          // %reg1032<def> = INSERT_SUBREG %reg1032, undef, 2
+          // This is an identity copy, eliminate it now.
+          if (MO.isKill()) {
+            LiveVariables::VarInfo& vi = lv_->getVarInfo(MO.getReg());
+            vi.removeKill(MI);
+          }
+          MI->eraseFromParent();
+          continue;
+        }
+      }
+
+      bool ChangedToImpDef = false;
+      for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
+        MachineOperand& MO = MI->getOperand(i);
+        if (!MO.isReg() || !MO.isUse() || MO.isUndef())
+          continue;
+        unsigned Reg = MO.getReg();
+        if (!Reg)
+          continue;
+        if (!ImpDefRegs.count(Reg))
+          continue;
+        // Use is a copy, just turn it into an implicit_def.
+        if (CanTurnIntoImplicitDef(MI, Reg, i, tii_)) {
+          bool isKill = MO.isKill();
+          MI->setDesc(tii_->get(TargetInstrInfo::IMPLICIT_DEF));
+          for (int j = MI->getNumOperands() - 1, ee = 0; j > ee; --j)
+            MI->RemoveOperand(j);
+          if (isKill) {
+            ImpDefRegs.erase(Reg);
+            LiveVariables::VarInfo& vi = lv_->getVarInfo(Reg);
+            vi.removeKill(MI);
+          }
+          ChangedToImpDef = true;
+          break;
+        }
+
+        MO.setIsUndef();
+        if (MO.isKill() || MI->isRegTiedToDefOperand(i)) {
+          // Make sure other uses of 
+          for (unsigned j = i+1; j != e; ++j) {
+            MachineOperand &MOJ = MI->getOperand(j);
+            if (MOJ.isReg() && MOJ.isUse() && MOJ.getReg() == Reg)
+              MOJ.setIsUndef();
+          }
+          ImpDefRegs.erase(Reg);
+        }
+      }
+
+      if (ChangedToImpDef) {
+        // Backtrack to process this new implicit_def.
+        --I;
+      } else {
+        for (unsigned i = 0; i != MI->getNumOperands(); ++i) {
+          MachineOperand& MO = MI->getOperand(i);
+          if (!MO.isReg() || !MO.isDef())
+            continue;
+          ImpDefRegs.erase(MO.getReg());
+        }
+      }
+    }
+
+    // Any outstanding liveout implicit_def's?
+    for (unsigned i = 0, e = ImpDefMIs.size(); i != e; ++i) {
+      MachineInstr *MI = ImpDefMIs[i];
+      unsigned Reg = MI->getOperand(0).getReg();
+      if (TargetRegisterInfo::isPhysicalRegister(Reg) ||
+          !ImpDefRegs.count(Reg)) {
+        // Delete all "local" implicit_def's. That include those which define
+        // physical registers since they cannot be liveout.
+        MI->eraseFromParent();
+        continue;
+      }
+
+      // If there are multiple defs of the same register and at least one
+      // is not an implicit_def, do not insert implicit_def's before the
+      // uses.
+      bool Skip = false;
+      for (MachineRegisterInfo::def_iterator DI = mri_->def_begin(Reg),
+             DE = mri_->def_end(); DI != DE; ++DI) {
+        if (DI->getOpcode() != TargetInstrInfo::IMPLICIT_DEF) {
+          Skip = true;
+          break;
+        }
+      }
+      if (Skip)
+        continue;
+
+      // The only implicit_def which we want to keep are those that are live
+      // out of its block.
+      MI->eraseFromParent();
+
+      for (MachineRegisterInfo::use_iterator UI = mri_->use_begin(Reg),
+             UE = mri_->use_end(); UI != UE; ) {
+        MachineOperand &RMO = UI.getOperand();
+        MachineInstr *RMI = &*UI;
+        ++UI;
+        MachineBasicBlock *RMBB = RMI->getParent();
+        if (RMBB == MBB)
+          continue;
+
+        // Turn a copy use into an implicit_def.
+        unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
+        if (tii_->isMoveInstr(*RMI, SrcReg, DstReg, SrcSubReg, DstSubReg) &&
+            Reg == SrcReg) {
+          RMI->setDesc(tii_->get(TargetInstrInfo::IMPLICIT_DEF));
+          for (int j = RMI->getNumOperands() - 1, ee = 0; j > ee; --j)
+            RMI->RemoveOperand(j);
+          continue;
+        }
+
+        const TargetRegisterClass* RC = mri_->getRegClass(Reg);
+        unsigned NewVReg = mri_->createVirtualRegister(RC);
+        RMO.setReg(NewVReg);
+        RMO.setIsUndef();
+        RMO.setIsKill();
+      }
+    }
+    ImpDefRegs.clear();
+    ImpDefMIs.clear();
+  }
+}
+
+
  void LiveIntervals::computeNumbering() {
    Index2MiMap OldI2MI = i2miMap_;
    std::vector<IdxMBBPair> OldI2MBB = Idx2MBBMap;
  void LiveIntervals::computeNumbering() {
    Index2MiMap OldI2MI = i2miMap_;
    std::vector<IdxMBBPair> OldI2MBB = Idx2MBBMap;
@@ -105,44 +283,79 @@ void LiveIntervals::computeNumbering() {
    MBB2IdxMap.clear();
    mi2iMap_.clear();
    i2miMap_.clear();
    MBB2IdxMap.clear();
    mi2iMap_.clear();
    i2miMap_.clear();
+  terminatorGaps.clear();
+  phiJoinCopies.clear();
    
    FunctionSize = 0;
    
    // Number MachineInstrs and MachineBasicBlocks.
    // Initialize MBB indexes to a sentinal.
    
    FunctionSize = 0;
    
    // Number MachineInstrs and MachineBasicBlocks.
    // Initialize MBB indexes to a sentinal.
-  MBB2IdxMap.resize(mf_->getNumBlockIDs(), std::make_pair(~0U,~0U));
+  MBB2IdxMap.resize(mf_->getNumBlockIDs(),
+                    std::make_pair(MachineInstrIndex(),MachineInstrIndex()));
    
    
-  unsigned MIIndex = 0;
+  MachineInstrIndex MIIndex;
    for (MachineFunction::iterator MBB = mf_->begin(), E = mf_->end();
         MBB != E; ++MBB) {
    for (MachineFunction::iterator MBB = mf_->begin(), E = mf_->end();
         MBB != E; ++MBB) {
-    unsigned StartIdx = MIIndex;
+    MachineInstrIndex StartIdx = MIIndex;
  
      // Insert an empty slot at the beginning of each block.
  
      // Insert an empty slot at the beginning of each block.
-    MIIndex += InstrSlots::NUM;
+    MIIndex = getNextIndex(MIIndex);
      i2miMap_.push_back(0);
  
      for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
           I != E; ++I) {
      i2miMap_.push_back(0);
  
      for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
           I != E; ++I) {
+      
+      if (I == MBB->getFirstTerminator()) {
+        // Leave a gap for before terminators, this is where we will point
+        // PHI kills.
+        MachineInstrIndex tGap(true, MIIndex);
+        bool inserted =
+          terminatorGaps.insert(std::make_pair(&*MBB, tGap)).second;
+        assert(inserted && 
+               "Multiple 'first' terminators encountered during numbering.");
+        inserted = inserted; // Avoid compiler warning if assertions turned off.
+        i2miMap_.push_back(0);
+
+        MIIndex = getNextIndex(MIIndex);
+      }
+
        bool inserted = mi2iMap_.insert(std::make_pair(I, MIIndex)).second;
        assert(inserted && "multiple MachineInstr -> index mappings");
        inserted = true;
        i2miMap_.push_back(I);
        bool inserted = mi2iMap_.insert(std::make_pair(I, MIIndex)).second;
        assert(inserted && "multiple MachineInstr -> index mappings");
        inserted = true;
        i2miMap_.push_back(I);
-      MIIndex += InstrSlots::NUM;
+      MIIndex = getNextIndex(MIIndex);
        FunctionSize++;
        
        // Insert max(1, numdefs) empty slots after every instruction.
        unsigned Slots = I->getDesc().getNumDefs();
        if (Slots == 0)
          Slots = 1;
        FunctionSize++;
        
        // Insert max(1, numdefs) empty slots after every instruction.
        unsigned Slots = I->getDesc().getNumDefs();
        if (Slots == 0)
          Slots = 1;
-      MIIndex += InstrSlots::NUM * Slots;
-      while (Slots--)
+      while (Slots--) {
+        MIIndex = getNextIndex(MIIndex);
          i2miMap_.push_back(0);
          i2miMap_.push_back(0);
+      }
+
+    }
+  
+    if (MBB->getFirstTerminator() == MBB->end()) {
+      // Leave a gap for before terminators, this is where we will point
+      // PHI kills.
+      MachineInstrIndex tGap(true, MIIndex);
+      bool inserted =
+        terminatorGaps.insert(std::make_pair(&*MBB, tGap)).second;
+      assert(inserted && 
+             "Multiple 'first' terminators encountered during numbering.");
+      inserted = inserted; // Avoid compiler warning if assertions turned off.
+      i2miMap_.push_back(0);
+ 
+      MIIndex = getNextIndex(MIIndex);
      }
      
      // Set the MBB2IdxMap entry for this MBB.
      }
      
      // Set the MBB2IdxMap entry for this MBB.
-    MBB2IdxMap[MBB->getNumber()] = std::make_pair(StartIdx, MIIndex - 1);
+    MBB2IdxMap[MBB->getNumber()] = std::make_pair(StartIdx, getPrevSlot(MIIndex));
      Idx2MBBMap.push_back(std::make_pair(StartIdx, MBB));
    }
      Idx2MBBMap.push_back(std::make_pair(StartIdx, MBB));
    }
+
    std::sort(Idx2MBBMap.begin(), Idx2MBBMap.end(), Idx2MBBCompare());
    
    if (!OldI2MI.empty())
    std::sort(Idx2MBBMap.begin(), Idx2MBBMap.end(), Idx2MBBCompare());
    
    if (!OldI2MI.empty())
@@ -154,9 +367,9 @@ void LiveIntervals::computeNumbering() {
          // number, or our best guess at what it _should_ correspond to if the
          // original instruction has been erased.  This is either the following
          // instruction or its predecessor.
          // number, or our best guess at what it _should_ correspond to if the
          // original instruction has been erased.  This is either the following
          // instruction or its predecessor.
-        unsigned index = LI->start / InstrSlots::NUM;
-        unsigned offset = LI->start % InstrSlots::NUM;
-        if (offset == InstrSlots::LOAD) {
+        unsigned index = LI->start.getVecIndex();
+        MachineInstrIndex::Slot offset = LI->start.getSlot();
+        if (LI->start.isLoad()) {
            std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), LI->start);
            // Take the pair containing the index
            std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), LI->start);
            // Take the pair containing the index
@@ -165,29 +378,34 @@ void LiveIntervals::computeNumbering() {
            
            LI->start = getMBBStartIdx(J->second);
          } else {
            
            LI->start = getMBBStartIdx(J->second);
          } else {
-          LI->start = mi2iMap_[OldI2MI[index]] + offset;
+          LI->start = MachineInstrIndex(
+            MachineInstrIndex(mi2iMap_[OldI2MI[index]]), 
+                              (MachineInstrIndex::Slot)offset);
          }
          
          // Remap the ending index in the same way that we remapped the start,
          // except for the final step where we always map to the immediately
          // following instruction.
          }
          
          // Remap the ending index in the same way that we remapped the start,
          // except for the final step where we always map to the immediately
          // following instruction.
-        index = (LI->end - 1) / InstrSlots::NUM;
-        offset  = LI->end % InstrSlots::NUM;
-        if (offset == InstrSlots::LOAD) {
+        index = (getPrevSlot(LI->end)).getVecIndex();
+        offset  = LI->end.getSlot();
+        if (LI->end.isLoad()) {
            // VReg dies at end of block.
            std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), LI->end);
            --I;
            
            // VReg dies at end of block.
            std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), LI->end);
            --I;
            
-          LI->end = getMBBEndIdx(I->second) + 1;
+          LI->end = getNextSlot(getMBBEndIdx(I->second));
          } else {
            unsigned idx = index;
            while (index < OldI2MI.size() && !OldI2MI[index]) ++index;
            
            if (index != OldI2MI.size())
          } else {
            unsigned idx = index;
            while (index < OldI2MI.size() && !OldI2MI[index]) ++index;
            
            if (index != OldI2MI.size())
-            LI->end = mi2iMap_[OldI2MI[index]] + (idx == index ? offset : 0);
+            LI->end =
+              MachineInstrIndex(mi2iMap_[OldI2MI[index]],
+                (idx == index ? offset : MachineInstrIndex::LOAD));
            else
            else
-            LI->end = InstrSlots::NUM * i2miMap_.size();
+            LI->end =
+              MachineInstrIndex(MachineInstrIndex::NUM * i2miMap_.size());
          }
        }
        
          }
        }
        
@@ -198,10 +416,10 @@ void LiveIntervals::computeNumbering() {
          // Remap the VNInfo def index, which works the same as the
          // start indices above. VN's with special sentinel defs
          // don't need to be remapped.
          // Remap the VNInfo def index, which works the same as the
          // start indices above. VN's with special sentinel defs
          // don't need to be remapped.
-        if (vni->def != ~0U && vni->def != ~1U) {
-          unsigned index = vni->def / InstrSlots::NUM;
-          unsigned offset = vni->def % InstrSlots::NUM;
-          if (offset == InstrSlots::LOAD) {
+        if (vni->isDefAccurate() && !vni->isUnused()) {
+          unsigned index = vni->def.getVecIndex();
+          MachineInstrIndex::Slot offset = vni->def.getSlot();
+          if (vni->def.isLoad()) {
              std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), vni->def);
              // Take the pair containing the index
              std::vector<IdxMBBPair>::const_iterator I =
                    std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), vni->def);
              // Take the pair containing the index
@@ -210,25 +428,36 @@ void LiveIntervals::computeNumbering() {
            
              vni->def = getMBBStartIdx(J->second);
            } else {
            
              vni->def = getMBBStartIdx(J->second);
            } else {
-            vni->def = mi2iMap_[OldI2MI[index]] + offset;
+            vni->def = MachineInstrIndex(mi2iMap_[OldI2MI[index]], offset);
            }
          }
          
          // Remap the VNInfo kill indices, which works the same as
          // the end indices above.
          for (size_t i = 0; i < vni->kills.size(); ++i) {
            }
          }
          
          // Remap the VNInfo kill indices, which works the same as
          // the end indices above.
          for (size_t i = 0; i < vni->kills.size(); ++i) {
-          // PHI kills don't need to be remapped.
-          if (!vni->kills[i]) continue;
-          
-          unsigned index = (vni->kills[i]-1) / InstrSlots::NUM;
-          unsigned offset = vni->kills[i] % InstrSlots::NUM;
-          if (offset == InstrSlots::LOAD) {
-            std::vector<IdxMBBPair>::const_iterator I =
+          unsigned index = getPrevSlot(vni->kills[i]).getVecIndex();
+          MachineInstrIndex::Slot offset = vni->kills[i].getSlot();
+
+          if (vni->kills[i].isLoad()) {
+            assert("Value killed at a load slot.");
+            /*std::vector<IdxMBBPair>::const_iterator I =
               std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), vni->kills[i]);
              --I;
  
               std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), vni->kills[i]);
              --I;
  
-            vni->kills[i] = getMBBEndIdx(I->second);
+            vni->kills[i] = getMBBEndIdx(I->second);*/
            } else {
            } else {
+            if (vni->kills[i].isPHIIndex()) {
+              std::vector<IdxMBBPair>::const_iterator I =
+                std::lower_bound(OldI2MBB.begin(), OldI2MBB.end(), vni->kills[i]);
+              --I;
+              vni->kills[i] = terminatorGaps[I->second];  
+            } else {
+              assert(OldI2MI[index] != 0 &&
+                     "Kill refers to instruction not present in index maps.");
+              vni->kills[i] = MachineInstrIndex(mi2iMap_[OldI2MI[index]], offset);
+            }
+           
+            /*
              unsigned idx = index;
              while (index < OldI2MI.size() && !OldI2MI[index]) ++index;
              
              unsigned idx = index;
              while (index < OldI2MI.size() && !OldI2MI[index]) ++index;
              
@@ -237,12 +466,64 @@ void LiveIntervals::computeNumbering() {
                                (idx == index ? offset : 0);
              else
                vni->kills[i] = InstrSlots::NUM * i2miMap_.size();
                                (idx == index ? offset : 0);
              else
                vni->kills[i] = InstrSlots::NUM * i2miMap_.size();
+            */
            }
          }
        }
      }
  }
  
            }
          }
        }
      }
  }
  
+void LiveIntervals::scaleNumbering(int factor) {
+  // Need to
+  //  * scale MBB begin and end points
+  //  * scale all ranges.
+  //  * Update VNI structures.
+  //  * Scale instruction numberings 
+
+  // Scale the MBB indices.
+  Idx2MBBMap.clear();
+  for (MachineFunction::iterator MBB = mf_->begin(), MBBE = mf_->end();
+       MBB != MBBE; ++MBB) {
+    std::pair<MachineInstrIndex, MachineInstrIndex> &mbbIndices = MBB2IdxMap[MBB->getNumber()];
+    mbbIndices.first = mbbIndices.first.scale(factor);
+    mbbIndices.second = mbbIndices.second.scale(factor);
+    Idx2MBBMap.push_back(std::make_pair(mbbIndices.first, MBB)); 
+  }
+  std::sort(Idx2MBBMap.begin(), Idx2MBBMap.end(), Idx2MBBCompare());
+
+  // Scale terminator gaps.
+  for (DenseMap<MachineBasicBlock*, MachineInstrIndex>::iterator
+       TGI = terminatorGaps.begin(), TGE = terminatorGaps.end();
+       TGI != TGE; ++TGI) {
+    terminatorGaps[TGI->first] = TGI->second.scale(factor);
+  }
+
+  // Scale the intervals.
+  for (iterator LI = begin(), LE = end(); LI != LE; ++LI) {
+    LI->second->scaleNumbering(factor);
+  }
+
+  // Scale MachineInstrs.
+  Mi2IndexMap oldmi2iMap = mi2iMap_;
+  MachineInstrIndex highestSlot;
+  for (Mi2IndexMap::iterator MI = oldmi2iMap.begin(), ME = oldmi2iMap.end();
+       MI != ME; ++MI) {
+    MachineInstrIndex newSlot = MI->second.scale(factor);
+    mi2iMap_[MI->first] = newSlot;
+    highestSlot = std::max(highestSlot, newSlot); 
+  }
+
+  unsigned highestVIndex = highestSlot.getVecIndex();
+  i2miMap_.clear();
+  i2miMap_.resize(highestVIndex + 1);
+  for (Mi2IndexMap::iterator MI = mi2iMap_.begin(), ME = mi2iMap_.end();
+       MI != ME; ++MI) {
+    i2miMap_[MI->second.getVecIndex()] = const_cast<MachineInstr *>(MI->first);
+  }
+
+}
+
+
  /// runOnMachineFunction - Register allocate the whole function
  ///
  bool LiveIntervals::runOnMachineFunction(MachineFunction &fn) {
  /// runOnMachineFunction - Register allocate the whole function
  ///
  bool LiveIntervals::runOnMachineFunction(MachineFunction &fn) {
@@ -255,8 +536,10 @@ bool LiveIntervals::runOnMachineFunction(MachineFunction &fn) {
    lv_ = &getAnalysis<LiveVariables>();
    allocatableRegs_ = tri_->getAllocatableSet(fn);
  
    lv_ = &getAnalysis<LiveVariables>();
    allocatableRegs_ = tri_->getAllocatableSet(fn);
  
+  processImplicitDefs();
    computeNumbering();
    computeIntervals();
    computeNumbering();
    computeIntervals();
+  performEarlyCoalescing();
  
    numIntervals += getNumIntervals();
  
  
    numIntervals += getNumIntervals();
  
@@ -265,36 +548,45 @@ bool LiveIntervals::runOnMachineFunction(MachineFunction &fn) {
  }
  
  /// print - Implement the dump method.
  }
  
  /// print - Implement the dump method.
-void LiveIntervals::print(std::ostream &O, const Module* ) const {
-  O << "********** INTERVALS **********\n";
+void LiveIntervals::print(raw_ostream &OS, const Module* ) const {
+  OS << "********** INTERVALS **********\n";
    for (const_iterator I = begin(), E = end(); I != E; ++I) {
    for (const_iterator I = begin(), E = end(); I != E; ++I) {
-    I->second->print(O, tri_);
-    O << "\n";
+    I->second->print(OS, tri_);
+    OS << "\n";
    }
  
    }
  
-  O << "********** MACHINEINSTRS **********\n";
+  printInstrs(OS);
+}
+
+void LiveIntervals::printInstrs(raw_ostream &OS) const {
+  OS << "********** MACHINEINSTRS **********\n";
+
    for (MachineFunction::iterator mbbi = mf_->begin(), mbbe = mf_->end();
         mbbi != mbbe; ++mbbi) {
    for (MachineFunction::iterator mbbi = mf_->begin(), mbbe = mf_->end();
         mbbi != mbbe; ++mbbi) {
-    O << ((Value*)mbbi->getBasicBlock())->getName() << ":\n";
+    OS << ((Value*)mbbi->getBasicBlock())->getName() << ":\n";
      for (MachineBasicBlock::iterator mii = mbbi->begin(),
             mie = mbbi->end(); mii != mie; ++mii) {
      for (MachineBasicBlock::iterator mii = mbbi->begin(),
             mie = mbbi->end(); mii != mie; ++mii) {
-      O << getInstructionIndex(mii) << '\t' << *mii;
+      OS << getInstructionIndex(mii) << '\t' << *mii;
      }
    }
  }
  
      }
    }
  }
  
+void LiveIntervals::dumpInstrs() const {
+  printInstrs(errs());
+}
+
  /// conflictsWithPhysRegDef - Returns true if the specified register
  /// is defined during the duration of the specified interval.
  bool LiveIntervals::conflictsWithPhysRegDef(const LiveInterval &li,
                                              VirtRegMap &vrm, unsigned reg) {
    for (LiveInterval::Ranges::const_iterator
           I = li.ranges.begin(), E = li.ranges.end(); I != E; ++I) {
  /// conflictsWithPhysRegDef - Returns true if the specified register
  /// is defined during the duration of the specified interval.
  bool LiveIntervals::conflictsWithPhysRegDef(const LiveInterval &li,
                                              VirtRegMap &vrm, unsigned reg) {
    for (LiveInterval::Ranges::const_iterator
           I = li.ranges.begin(), E = li.ranges.end(); I != E; ++I) {
-    for (unsigned index = getBaseIndex(I->start),
-           end = getBaseIndex(I->end-1) + InstrSlots::NUM; index != end;
-         index += InstrSlots::NUM) {
+    for (MachineInstrIndex index = getBaseIndex(I->start),
+           end = getNextIndex(getBaseIndex(getPrevSlot(I->end))); index != end;
+         index = getNextIndex(index)) {
        // skip deleted instructions
        while (index != end && !getInstructionFromIndex(index))
        // skip deleted instructions
        while (index != end && !getInstructionFromIndex(index))
-        index += InstrSlots::NUM;
+        index = getNextIndex(index);
        if (index == end) break;
  
        MachineInstr *MI = getInstructionFromIndex(index);
        if (index == end) break;
  
        MachineInstr *MI = getInstructionFromIndex(index);
@@ -330,16 +622,16 @@ bool LiveIntervals::conflictsWithPhysRegRef(LiveInterval &li,
                                    SmallPtrSet<MachineInstr*,32> &JoinedCopies) {
    for (LiveInterval::Ranges::const_iterator
           I = li.ranges.begin(), E = li.ranges.end(); I != E; ++I) {
                                    SmallPtrSet<MachineInstr*,32> &JoinedCopies) {
    for (LiveInterval::Ranges::const_iterator
           I = li.ranges.begin(), E = li.ranges.end(); I != E; ++I) {
-    for (unsigned index = getBaseIndex(I->start),
-           end = getBaseIndex(I->end-1) + InstrSlots::NUM; index != end;
-         index += InstrSlots::NUM) {
+    for (MachineInstrIndex index = getBaseIndex(I->start),
+           end = getNextIndex(getBaseIndex(getPrevSlot(I->end))); index != end;
+         index = getNextIndex(index)) {
        // Skip deleted instructions.
        MachineInstr *MI = 0;
        while (index != end) {
          MI = getInstructionFromIndex(index);
          if (MI)
            break;
        // Skip deleted instructions.
        MachineInstr *MI = 0;
        while (index != end) {
          MI = getInstructionFromIndex(index);
          if (MI)
            break;
-        index += InstrSlots::NUM;
+        index = getNextIndex(index);
        }
        if (index == end) break;
  
        }
        if (index == end) break;
  
@@ -363,35 +655,36 @@ bool LiveIntervals::conflictsWithPhysRegRef(LiveInterval &li,
    return false;
  }
  
    return false;
  }
  
-
-void LiveIntervals::printRegName(unsigned reg) const {
+#ifndef NDEBUG
+static void printRegName(unsigned reg, const TargetRegisterInfo* tri_) {
    if (TargetRegisterInfo::isPhysicalRegister(reg))
    if (TargetRegisterInfo::isPhysicalRegister(reg))
-    cerr << tri_->getName(reg);
+    errs() << tri_->getName(reg);
    else
    else
-    cerr << "%reg" << reg;
+    errs() << "%reg" << reg;
  }
  }
+#endif
  
  void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
                                               MachineBasicBlock::iterator mi,
  
  void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
                                               MachineBasicBlock::iterator mi,
-                                             unsigned MIIdx, MachineOperand& MO,
+                                             MachineInstrIndex MIIdx,
+                                             MachineOperand& MO,
                                               unsigned MOIdx,
                                               LiveInterval &interval) {
                                               unsigned MOIdx,
                                               LiveInterval &interval) {
-  DOUT << "\t\tregister: "; DEBUG(printRegName(interval.reg));
-  LiveVariables::VarInfo& vi = lv_->getVarInfo(interval.reg);
-
-  if (mi->getOpcode() == TargetInstrInfo::IMPLICIT_DEF) {
-    DOUT << "is a implicit_def\n";
-    return;
-  }
+  DEBUG({
+      errs() << "\t\tregister: ";
+      printRegName(interval.reg, tri_);
+    });
  
    // Virtual registers may be defined multiple times (due to phi
    // elimination and 2-addr elimination).  Much of what we do only has to be
    // done once for the vreg.  We use an empty interval to detect the first
    // time we see a vreg.
  
    // Virtual registers may be defined multiple times (due to phi
    // elimination and 2-addr elimination).  Much of what we do only has to be
    // done once for the vreg.  We use an empty interval to detect the first
    // time we see a vreg.
+  LiveVariables::VarInfo& vi = lv_->getVarInfo(interval.reg);
    if (interval.empty()) {
      // Get the Idx of the defining instructions.
    if (interval.empty()) {
      // Get the Idx of the defining instructions.
-    unsigned defIndex = getDefIndex(MIIdx);
-    // Earlyclobbers move back one.
+    MachineInstrIndex defIndex = getDefIndex(MIIdx);
+    // Earlyclobbers move back one, so that they overlap the live range
+    // of inputs.
      if (MO.isEarlyClobber())
        defIndex = getUseIndex(MIIdx);
      VNInfo *ValNo;
      if (MO.isEarlyClobber())
        defIndex = getUseIndex(MIIdx);
      VNInfo *ValNo;
@@ -403,7 +696,7 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
          tii_->isMoveInstr(*mi, SrcReg, DstReg, SrcSubReg, DstSubReg))
        CopyMI = mi;
      // Earlyclobbers move back one.
          tii_->isMoveInstr(*mi, SrcReg, DstReg, SrcSubReg, DstSubReg))
        CopyMI = mi;
      // Earlyclobbers move back one.
-    ValNo = interval.getNextValue(defIndex, CopyMI, VNInfoAllocator);
+    ValNo = interval.getNextValue(defIndex, CopyMI, true, VNInfoAllocator);
  
      assert(ValNo->id == 0 && "First value in interval is not 0?");
  
  
      assert(ValNo->id == 0 && "First value in interval is not 0?");
  
@@ -413,21 +706,26 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
      // will be a single kill, in MBB, which comes after the definition.
      if (vi.Kills.size() == 1 && vi.Kills[0]->getParent() == mbb) {
        // FIXME: what about dead vars?
      // will be a single kill, in MBB, which comes after the definition.
      if (vi.Kills.size() == 1 && vi.Kills[0]->getParent() == mbb) {
        // FIXME: what about dead vars?
-      unsigned killIdx;
+      MachineInstrIndex killIdx;
        if (vi.Kills[0] != mi)
        if (vi.Kills[0] != mi)
-        killIdx = getUseIndex(getInstructionIndex(vi.Kills[0]))+1;
+        killIdx = getNextSlot(getUseIndex(getInstructionIndex(vi.Kills[0])));
+      else if (MO.isEarlyClobber())
+        // Earlyclobbers that die in this instruction move up one extra, to
+        // compensate for having the starting point moved back one.  This
+        // gets them to overlap the live range of other outputs.
+        killIdx = getNextSlot(getNextSlot(defIndex));
        else
        else
-        killIdx = defIndex+1;
+        killIdx = getNextSlot(defIndex);
  
        // If the kill happens after the definition, we have an intra-block
        // live range.
        if (killIdx > defIndex) {
  
        // If the kill happens after the definition, we have an intra-block
        // live range.
        if (killIdx > defIndex) {
-        assert(vi.AliveBlocks.none() &&
+        assert(vi.AliveBlocks.empty() &&
                 "Shouldn't be alive across any blocks!");
          LiveRange LR(defIndex, killIdx, ValNo);
          interval.addRange(LR);
                 "Shouldn't be alive across any blocks!");
          LiveRange LR(defIndex, killIdx, ValNo);
          interval.addRange(LR);
-        DOUT << " +" << LR << "\n";
-        interval.addKill(ValNo, killIdx);
+        DEBUG(errs() << " +" << LR << "\n");
+        ValNo->addKill(killIdx);
          return;
        }
      }
          return;
        }
      }
@@ -436,32 +734,32 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
      // of the defining block, potentially live across some blocks, then is
      // live into some number of blocks, but gets killed.  Start by adding a
      // range that goes from this definition to the end of the defining block.
      // of the defining block, potentially live across some blocks, then is
      // live into some number of blocks, but gets killed.  Start by adding a
      // range that goes from this definition to the end of the defining block.
-    LiveRange NewLR(defIndex, getMBBEndIdx(mbb)+1, ValNo);
-    DOUT << " +" << NewLR;
+    LiveRange NewLR(defIndex, getNextSlot(getMBBEndIdx(mbb)), ValNo);
+    DEBUG(errs() << " +" << NewLR);
      interval.addRange(NewLR);
  
      // Iterate over all of the blocks that the variable is completely
      // live in, adding [insrtIndex(begin), instrIndex(end)+4) to the
      // live interval.
      interval.addRange(NewLR);
  
      // Iterate over all of the blocks that the variable is completely
      // live in, adding [insrtIndex(begin), instrIndex(end)+4) to the
      // live interval.
-    for (int i = vi.AliveBlocks.find_first(); i != -1;
-         i = vi.AliveBlocks.find_next(i)) {
-      LiveRange LR(getMBBStartIdx(i),
-                   getMBBEndIdx(i)+1,  // MBB ends at -1.
+    for (SparseBitVector<>::iterator I = vi.AliveBlocks.begin(), 
+             E = vi.AliveBlocks.end(); I != E; ++I) {
+      LiveRange LR(getMBBStartIdx(*I),
+                   getNextSlot(getMBBEndIdx(*I)),  // MBB ends at -1.
                     ValNo);
        interval.addRange(LR);
                     ValNo);
        interval.addRange(LR);
-      DOUT << " +" << LR;
+      DEBUG(errs() << " +" << LR);
      }
  
      // Finally, this virtual register is live from the start of any killing
      // block to the 'use' slot of the killing instruction.
      for (unsigned i = 0, e = vi.Kills.size(); i != e; ++i) {
        MachineInstr *Kill = vi.Kills[i];
      }
  
      // Finally, this virtual register is live from the start of any killing
      // block to the 'use' slot of the killing instruction.
      for (unsigned i = 0, e = vi.Kills.size(); i != e; ++i) {
        MachineInstr *Kill = vi.Kills[i];
-      unsigned killIdx = getUseIndex(getInstructionIndex(Kill))+1;
-      LiveRange LR(getMBBStartIdx(Kill->getParent()),
-                   killIdx, ValNo);
+      MachineInstrIndex killIdx =
+        getNextSlot(getUseIndex(getInstructionIndex(Kill)));
+      LiveRange LR(getMBBStartIdx(Kill->getParent()), killIdx, ValNo);
        interval.addRange(LR);
        interval.addRange(LR);
-      interval.addKill(ValNo, killIdx);
-      DOUT << " +" << LR;
+      ValNo->addKill(killIdx);
+      DEBUG(errs() << " +" << LR);
      }
  
    } else {
      }
  
    } else {
@@ -469,19 +767,20 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
      // must be due to phi elimination or two addr elimination.  If this is
      // the result of two address elimination, then the vreg is one of the
      // def-and-use register operand.
      // must be due to phi elimination or two addr elimination.  If this is
      // the result of two address elimination, then the vreg is one of the
      // def-and-use register operand.
-    if (mi->isRegReDefinedByTwoAddr(MOIdx)) {
+    if (mi->isRegTiedToUseOperand(MOIdx)) {
        // If this is a two-address definition, then we have already processed
        // the live range.  The only problem is that we didn't realize there
        // are actually two values in the live interval.  Because of this we
        // need to take the LiveRegion that defines this register and split it
        // into two values.
        assert(interval.containsOneValue());
        // If this is a two-address definition, then we have already processed
        // the live range.  The only problem is that we didn't realize there
        // are actually two values in the live interval.  Because of this we
        // need to take the LiveRegion that defines this register and split it
        // into two values.
        assert(interval.containsOneValue());
-      unsigned DefIndex = getDefIndex(interval.getValNumInfo(0)->def);
-      unsigned RedefIndex = getDefIndex(MIIdx);
+      MachineInstrIndex DefIndex = getDefIndex(interval.getValNumInfo(0)->def);
+      MachineInstrIndex RedefIndex = getDefIndex(MIIdx);
        if (MO.isEarlyClobber())
          RedefIndex = getUseIndex(MIIdx);
  
        if (MO.isEarlyClobber())
          RedefIndex = getUseIndex(MIIdx);
  
-      const LiveRange *OldLR = interval.getLiveRangeContaining(RedefIndex-1);
+      const LiveRange *OldLR =
+        interval.getLiveRangeContaining(getPrevSlot(RedefIndex));
        VNInfo *OldValNo = OldLR->valno;
  
        // Delete the initial value, which should be short and continuous,
        VNInfo *OldValNo = OldLR->valno;
  
        // Delete the initial value, which should be short and continuous,
@@ -494,64 +793,85 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
  
        // The new value number (#1) is defined by the instruction we claimed
        // defined value #0.
  
        // The new value number (#1) is defined by the instruction we claimed
        // defined value #0.
-      VNInfo *ValNo = interval.getNextValue(OldValNo->def, OldValNo->copy,
+      VNInfo *ValNo = interval.getNextValue(OldValNo->def, OldValNo->getCopy(),
+                                            false, // update at *
                                              VNInfoAllocator);
                                              VNInfoAllocator);
-      
+      ValNo->setFlags(OldValNo->getFlags()); // * <- updating here
+
        // Value#0 is now defined by the 2-addr instruction.
        OldValNo->def  = RedefIndex;
        // Value#0 is now defined by the 2-addr instruction.
        OldValNo->def  = RedefIndex;
-      OldValNo->copy = 0;
+      OldValNo->setCopy(0);
        if (MO.isEarlyClobber())
        if (MO.isEarlyClobber())
-        OldValNo->redefByEC = true;
+        OldValNo->setHasRedefByEC(true);
        
        // Add the new live interval which replaces the range for the input copy.
        LiveRange LR(DefIndex, RedefIndex, ValNo);
        
        // Add the new live interval which replaces the range for the input copy.
        LiveRange LR(DefIndex, RedefIndex, ValNo);
-      DOUT << " replace range with " << LR;
+      DEBUG(errs() << " replace range with " << LR);
        interval.addRange(LR);
        interval.addRange(LR);
-      interval.addKill(ValNo, RedefIndex);
+      ValNo->addKill(RedefIndex);
  
        // If this redefinition is dead, we need to add a dummy unit live
        // range covering the def slot.
        if (MO.isDead())
  
        // If this redefinition is dead, we need to add a dummy unit live
        // range covering the def slot.
        if (MO.isDead())
-        interval.addRange(LiveRange(RedefIndex, RedefIndex+1, OldValNo));
-
-      DOUT << " RESULT: ";
-      interval.print(DOUT, tri_);
-
+        interval.addRange(
+          LiveRange(RedefIndex, MO.isEarlyClobber() ?
+                                getNextSlot(getNextSlot(RedefIndex)) :
+                                getNextSlot(RedefIndex), OldValNo));
+
+      DEBUG({
+          errs() << " RESULT: ";
+          interval.print(errs(), tri_);
+        });
      } else {
        // Otherwise, this must be because of phi elimination.  If this is the
        // first redefinition of the vreg that we have seen, go back and change
        // the live range in the PHI block to be a different value number.
        if (interval.containsOneValue()) {
      } else {
        // Otherwise, this must be because of phi elimination.  If this is the
        // first redefinition of the vreg that we have seen, go back and change
        // the live range in the PHI block to be a different value number.
        if (interval.containsOneValue()) {
-        assert(vi.Kills.size() == 1 &&
-               "PHI elimination vreg should have one kill, the PHI itself!");
-
          // Remove the old range that we now know has an incorrect number.
          VNInfo *VNI = interval.getValNumInfo(0);
          MachineInstr *Killer = vi.Kills[0];
          // Remove the old range that we now know has an incorrect number.
          VNInfo *VNI = interval.getValNumInfo(0);
          MachineInstr *Killer = vi.Kills[0];
-        unsigned Start = getMBBStartIdx(Killer->getParent());
-        unsigned End = getUseIndex(getInstructionIndex(Killer))+1;
-        DOUT << " Removing [" << Start << "," << End << "] from: ";
-        interval.print(DOUT, tri_); DOUT << "\n";
-        interval.removeRange(Start, End);
-        VNI->hasPHIKill = true;
-        DOUT << " RESULT: "; interval.print(DOUT, tri_);
+        phiJoinCopies.push_back(Killer);
+        MachineInstrIndex Start = getMBBStartIdx(Killer->getParent());
+        MachineInstrIndex End =
+          getNextSlot(getUseIndex(getInstructionIndex(Killer)));
+        DEBUG({
+            errs() << " Removing [" << Start << "," << End << "] from: ";
+            interval.print(errs(), tri_);
+            errs() << "\n";
+          });
+        interval.removeRange(Start, End);        
+        assert(interval.ranges.size() == 1 &&
+               "Newly discovered PHI interval has >1 ranges.");
+        MachineBasicBlock *killMBB = getMBBFromIndex(interval.endIndex());
+        VNI->addKill(terminatorGaps[killMBB]);
+        VNI->setHasPHIKill(true);
+        DEBUG({
+            errs() << " RESULT: ";
+            interval.print(errs(), tri_);
+          });
  
          // Replace the interval with one of a NEW value number.  Note that this
          // value number isn't actually defined by an instruction, weird huh? :)
  
          // Replace the interval with one of a NEW value number.  Note that this
          // value number isn't actually defined by an instruction, weird huh? :)
-        LiveRange LR(Start, End, interval.getNextValue(~0, 0, VNInfoAllocator));
-        DOUT << " replace range with " << LR;
+        LiveRange LR(Start, End,
+          interval.getNextValue(MachineInstrIndex(mbb->getNumber()),
+                                0, false, VNInfoAllocator));
+        LR.valno->setIsPHIDef(true);
+        DEBUG(errs() << " replace range with " << LR);
          interval.addRange(LR);
          interval.addRange(LR);
-        interval.addKill(LR.valno, End);
-        DOUT << " RESULT: "; interval.print(DOUT, tri_);
+        LR.valno->addKill(End);
+        DEBUG({
+            errs() << " RESULT: ";
+            interval.print(errs(), tri_);
+          });
        }
  
        // In the case of PHI elimination, each variable definition is only
        // live until the end of the block.  We've already taken care of the
        // rest of the live range.
        }
  
        // In the case of PHI elimination, each variable definition is only
        // live until the end of the block.  We've already taken care of the
        // rest of the live range.
-      unsigned defIndex = getDefIndex(MIIdx);
+      MachineInstrIndex defIndex = getDefIndex(MIIdx);
        if (MO.isEarlyClobber())
          defIndex = getUseIndex(MIIdx);
        if (MO.isEarlyClobber())
          defIndex = getUseIndex(MIIdx);
-      
+
        VNInfo *ValNo;
        MachineInstr *CopyMI = NULL;
        unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
        VNInfo *ValNo;
        MachineInstr *CopyMI = NULL;
        unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
@@ -560,76 +880,94 @@ void LiveIntervals::handleVirtualRegisterDef(MachineBasicBlock *mbb,
            mi->getOpcode() == TargetInstrInfo::SUBREG_TO_REG ||
            tii_->isMoveInstr(*mi, SrcReg, DstReg, SrcSubReg, DstSubReg))
          CopyMI = mi;
            mi->getOpcode() == TargetInstrInfo::SUBREG_TO_REG ||
            tii_->isMoveInstr(*mi, SrcReg, DstReg, SrcSubReg, DstSubReg))
          CopyMI = mi;
-      ValNo = interval.getNextValue(defIndex, CopyMI, VNInfoAllocator);
+      ValNo = interval.getNextValue(defIndex, CopyMI, true, VNInfoAllocator);
        
        
-      unsigned killIndex = getMBBEndIdx(mbb) + 1;
+      MachineInstrIndex killIndex = getNextSlot(getMBBEndIdx(mbb));
        LiveRange LR(defIndex, killIndex, ValNo);
        interval.addRange(LR);
        LiveRange LR(defIndex, killIndex, ValNo);
        interval.addRange(LR);
-      interval.addKill(ValNo, killIndex);
-      ValNo->hasPHIKill = true;
-      DOUT << " +" << LR;
+      ValNo->addKill(terminatorGaps[mbb]);
+      ValNo->setHasPHIKill(true);
+      DEBUG(errs() << " +" << LR);
      }
    }
  
      }
    }
  
-  DOUT << '\n';
+  DEBUG(errs() << '\n');
  }
  
  void LiveIntervals::handlePhysicalRegisterDef(MachineBasicBlock *MBB,
                                                MachineBasicBlock::iterator mi,
  }
  
  void LiveIntervals::handlePhysicalRegisterDef(MachineBasicBlock *MBB,
                                                MachineBasicBlock::iterator mi,
-                                              unsigned MIIdx,
+                                              MachineInstrIndex MIIdx,
                                                MachineOperand& MO,
                                                LiveInterval &interval,
                                                MachineInstr *CopyMI) {
    // A physical register cannot be live across basic block, so its
    // lifetime must end somewhere in its defining basic block.
                                                MachineOperand& MO,
                                                LiveInterval &interval,
                                                MachineInstr *CopyMI) {
    // A physical register cannot be live across basic block, so its
    // lifetime must end somewhere in its defining basic block.
-  DOUT << "\t\tregister: "; DEBUG(printRegName(interval.reg));
+  DEBUG({
+      errs() << "\t\tregister: ";
+      printRegName(interval.reg, tri_);
+    });
  
  
-  unsigned baseIndex = MIIdx;
-  unsigned start = getDefIndex(baseIndex);
+  MachineInstrIndex baseIndex = MIIdx;
+  MachineInstrIndex start = getDefIndex(baseIndex);
    // Earlyclobbers move back one.
    if (MO.isEarlyClobber())
      start = getUseIndex(MIIdx);
    // Earlyclobbers move back one.
    if (MO.isEarlyClobber())
      start = getUseIndex(MIIdx);
-  unsigned end = start;
+  MachineInstrIndex end = start;
  
    // If it is not used after definition, it is considered dead at
    // the instruction defining it. Hence its interval is:
    // [defSlot(def), defSlot(def)+1)
  
    // If it is not used after definition, it is considered dead at
    // the instruction defining it. Hence its interval is:
    // [defSlot(def), defSlot(def)+1)
+  // For earlyclobbers, the defSlot was pushed back one; the extra
+  // advance below compensates.
    if (MO.isDead()) {
    if (MO.isDead()) {
-    DOUT << " dead";
-    end = start + 1;
+    DEBUG(errs() << " dead");
+    if (MO.isEarlyClobber())
+      end = getNextSlot(getNextSlot(start));
+    else
+      end = getNextSlot(start);
      goto exit;
    }
  
    // If it is not dead on definition, it must be killed by a
    // subsequent instruction. Hence its interval is:
    // [defSlot(def), useSlot(kill)+1)
      goto exit;
    }
  
    // If it is not dead on definition, it must be killed by a
    // subsequent instruction. Hence its interval is:
    // [defSlot(def), useSlot(kill)+1)
-  baseIndex += InstrSlots::NUM;
+  baseIndex = getNextIndex(baseIndex);
    while (++mi != MBB->end()) {
    while (++mi != MBB->end()) {
-    while (baseIndex / InstrSlots::NUM < i2miMap_.size() &&
+    while (baseIndex.getVecIndex() < i2miMap_.size() &&
             getInstructionFromIndex(baseIndex) == 0)
             getInstructionFromIndex(baseIndex) == 0)
-      baseIndex += InstrSlots::NUM;
+      baseIndex = getNextIndex(baseIndex);
      if (mi->killsRegister(interval.reg, tri_)) {
      if (mi->killsRegister(interval.reg, tri_)) {
-      DOUT << " killed";
-      end = getUseIndex(baseIndex) + 1;
-      goto exit;
-    } else if (mi->modifiesRegister(interval.reg, tri_)) {
-      // Another instruction redefines the register before it is ever read.
-      // Then the register is essentially dead at the instruction that defines
-      // it. Hence its interval is:
-      // [defSlot(def), defSlot(def)+1)
-      DOUT << " dead";
-      end = start + 1;
+      DEBUG(errs() << " killed");
+      end = getNextSlot(getUseIndex(baseIndex));
        goto exit;
        goto exit;
+    } else {
+      int DefIdx = mi->findRegisterDefOperandIdx(interval.reg, false, tri_);
+      if (DefIdx != -1) {
+        if (mi->isRegTiedToUseOperand(DefIdx)) {
+          // Two-address instruction.
+          end = getDefIndex(baseIndex);
+          if (mi->getOperand(DefIdx).isEarlyClobber())
+            end = getUseIndex(baseIndex);
+        } else {
+          // Another instruction redefines the register before it is ever read.
+          // Then the register is essentially dead at the instruction that defines
+          // it. Hence its interval is:
+          // [defSlot(def), defSlot(def)+1)
+          DEBUG(errs() << " dead");
+          end = getNextSlot(start);
+        }
+        goto exit;
+      }
      }
      
      }
      
-    baseIndex += InstrSlots::NUM;
+    baseIndex = getNextIndex(baseIndex);
    }
    
    // The only case we should have a dead physreg here without a killing or
    // instruction where we know it's dead is if it is live-in to the function
    }
    
    // The only case we should have a dead physreg here without a killing or
    // instruction where we know it's dead is if it is live-in to the function
-  // and never used.
-  assert(!CopyMI && "physreg was not killed in defining block!");
-  end = start + 1;
+  // and never used. Another possible case is the implicit use of the
+  // physical register has been deleted by two-address pass.
+  end = getNextSlot(start);
  
  exit:
    assert(start < end && "did not find end of interval?");
  
  exit:
    assert(start < end && "did not find end of interval?");
@@ -638,18 +976,18 @@ exit:
    LiveInterval::iterator OldLR = interval.FindLiveRangeContaining(start);
    bool Extend = OldLR != interval.end();
    VNInfo *ValNo = Extend
    LiveInterval::iterator OldLR = interval.FindLiveRangeContaining(start);
    bool Extend = OldLR != interval.end();
    VNInfo *ValNo = Extend
-    ? OldLR->valno : interval.getNextValue(start, CopyMI, VNInfoAllocator);
+    ? OldLR->valno : interval.getNextValue(start, CopyMI, true, VNInfoAllocator);
    if (MO.isEarlyClobber() && Extend)
    if (MO.isEarlyClobber() && Extend)
-    ValNo->redefByEC = true;
+    ValNo->setHasRedefByEC(true);
    LiveRange LR(start, end, ValNo);
    interval.addRange(LR);
    LiveRange LR(start, end, ValNo);
    interval.addRange(LR);
-  interval.addKill(LR.valno, end);
-  DOUT << " +" << LR << '\n';
+  LR.valno->addKill(end);
+  DEBUG(errs() << " +" << LR << '\n');
  }
  
  void LiveIntervals::handleRegisterDef(MachineBasicBlock *MBB,
                                        MachineBasicBlock::iterator MI,
  }
  
  void LiveIntervals::handleRegisterDef(MachineBasicBlock *MBB,
                                        MachineBasicBlock::iterator MI,
-                                      unsigned MIIdx,
+                                      MachineInstrIndex MIIdx,
                                        MachineOperand& MO,
                                        unsigned MOIdx) {
    if (TargetRegisterInfo::isVirtualRegister(MO.getReg()))
                                        MachineOperand& MO,
                                        unsigned MOIdx) {
    if (TargetRegisterInfo::isVirtualRegister(MO.getReg()))
@@ -663,76 +1001,213 @@ void LiveIntervals::handleRegisterDef(MachineBasicBlock *MBB,
          MI->getOpcode() == TargetInstrInfo::SUBREG_TO_REG ||
          tii_->isMoveInstr(*MI, SrcReg, DstReg, SrcSubReg, DstSubReg))
        CopyMI = MI;
          MI->getOpcode() == TargetInstrInfo::SUBREG_TO_REG ||
          tii_->isMoveInstr(*MI, SrcReg, DstReg, SrcSubReg, DstSubReg))
        CopyMI = MI;
-    handlePhysicalRegisterDef(MBB, MI, MIIdx, MO, 
+    handlePhysicalRegisterDef(MBB, MI, MIIdx, MO,
                                getOrCreateInterval(MO.getReg()), CopyMI);
      // Def of a register also defines its sub-registers.
      for (const unsigned* AS = tri_->getSubRegisters(MO.getReg()); *AS; ++AS)
        // If MI also modifies the sub-register explicitly, avoid processing it
        // more than once. Do not pass in TRI here so it checks for exact match.
        if (!MI->modifiesRegister(*AS))
                                getOrCreateInterval(MO.getReg()), CopyMI);
      // Def of a register also defines its sub-registers.
      for (const unsigned* AS = tri_->getSubRegisters(MO.getReg()); *AS; ++AS)
        // If MI also modifies the sub-register explicitly, avoid processing it
        // more than once. Do not pass in TRI here so it checks for exact match.
        if (!MI->modifiesRegister(*AS))
-        handlePhysicalRegisterDef(MBB, MI, MIIdx, MO, 
+        handlePhysicalRegisterDef(MBB, MI, MIIdx, MO,
                                    getOrCreateInterval(*AS), 0);
    }
  }
  
  void LiveIntervals::handleLiveInRegister(MachineBasicBlock *MBB,
                                    getOrCreateInterval(*AS), 0);
    }
  }
  
  void LiveIntervals::handleLiveInRegister(MachineBasicBlock *MBB,
-                                         unsigned MIIdx,
+                                         MachineInstrIndex MIIdx,
                                           LiveInterval &interval, bool isAlias) {
                                           LiveInterval &interval, bool isAlias) {
-  DOUT << "\t\tlivein register: "; DEBUG(printRegName(interval.reg));
+  DEBUG({
+      errs() << "\t\tlivein register: ";
+      printRegName(interval.reg, tri_);
+    });
  
    // Look for kills, if it reaches a def before it's killed, then it shouldn't
    // be considered a livein.
    MachineBasicBlock::iterator mi = MBB->begin();
  
    // Look for kills, if it reaches a def before it's killed, then it shouldn't
    // be considered a livein.
    MachineBasicBlock::iterator mi = MBB->begin();
-  unsigned baseIndex = MIIdx;
-  unsigned start = baseIndex;
-  while (baseIndex / InstrSlots::NUM < i2miMap_.size() && 
+  MachineInstrIndex baseIndex = MIIdx;
+  MachineInstrIndex start = baseIndex;
+  while (baseIndex.getVecIndex() < i2miMap_.size() && 
           getInstructionFromIndex(baseIndex) == 0)
           getInstructionFromIndex(baseIndex) == 0)
-    baseIndex += InstrSlots::NUM;
-  unsigned end = baseIndex;
+    baseIndex = getNextIndex(baseIndex);
+  MachineInstrIndex end = baseIndex;
    bool SeenDefUse = false;
    
    while (mi != MBB->end()) {
      if (mi->killsRegister(interval.reg, tri_)) {
    bool SeenDefUse = false;
    
    while (mi != MBB->end()) {
      if (mi->killsRegister(interval.reg, tri_)) {
-      DOUT << " killed";
-      end = getUseIndex(baseIndex) + 1;
+      DEBUG(errs() << " killed");
+      end = getNextSlot(getUseIndex(baseIndex));
        SeenDefUse = true;
        SeenDefUse = true;
-      goto exit;
+      break;
      } else if (mi->modifiesRegister(interval.reg, tri_)) {
        // Another instruction redefines the register before it is ever read.
        // Then the register is essentially dead at the instruction that defines
        // it. Hence its interval is:
        // [defSlot(def), defSlot(def)+1)
      } else if (mi->modifiesRegister(interval.reg, tri_)) {
        // Another instruction redefines the register before it is ever read.
        // Then the register is essentially dead at the instruction that defines
        // it. Hence its interval is:
        // [defSlot(def), defSlot(def)+1)
-      DOUT << " dead";
-      end = getDefIndex(start) + 1;
+      DEBUG(errs() << " dead");
+      end = getNextSlot(getDefIndex(start));
        SeenDefUse = true;
        SeenDefUse = true;
-      goto exit;
+      break;
      }
  
      }
  
-    baseIndex += InstrSlots::NUM;
+    baseIndex = getNextIndex(baseIndex);
      ++mi;
      if (mi != MBB->end()) {
      ++mi;
      if (mi != MBB->end()) {
-      while (baseIndex / InstrSlots::NUM < i2miMap_.size() && 
+      while (baseIndex.getVecIndex() < i2miMap_.size() && 
               getInstructionFromIndex(baseIndex) == 0)
               getInstructionFromIndex(baseIndex) == 0)
-        baseIndex += InstrSlots::NUM;
+        baseIndex = getNextIndex(baseIndex);
      }
    }
  
      }
    }
  
-exit:
    // Live-in register might not be used at all.
    if (!SeenDefUse) {
      if (isAlias) {
    // Live-in register might not be used at all.
    if (!SeenDefUse) {
      if (isAlias) {
-      DOUT << " dead";
-      end = getDefIndex(MIIdx) + 1;
+      DEBUG(errs() << " dead");
+      end = getNextSlot(getDefIndex(MIIdx));
      } else {
      } else {
-      DOUT << " live through";
+      DEBUG(errs() << " live through");
        end = baseIndex;
      }
    }
  
        end = baseIndex;
      }
    }
  
-  LiveRange LR(start, end, interval.getNextValue(~0U, 0, VNInfoAllocator));
+  VNInfo *vni =
+    interval.getNextValue(MachineInstrIndex(MBB->getNumber()),
+                          0, false, VNInfoAllocator);
+  vni->setIsPHIDef(true);
+  LiveRange LR(start, end, vni);
+  
    interval.addRange(LR);
    interval.addRange(LR);
-  interval.addKill(LR.valno, end);
-  DOUT << " +" << LR << '\n';
+  LR.valno->addKill(end);
+  DEBUG(errs() << " +" << LR << '\n');
+}
+
+bool
+LiveIntervals::isProfitableToCoalesce(LiveInterval &DstInt, LiveInterval &SrcInt,
+                                   SmallVector<MachineInstr*,16> &IdentCopies,
+                                   SmallVector<MachineInstr*,16> &OtherCopies) {
+  bool HaveConflict = false;
+  unsigned NumIdent = 0;
+  for (MachineRegisterInfo::reg_iterator ri = mri_->reg_begin(SrcInt.reg),
+         re = mri_->reg_end(); ri != re; ++ri) {
+    MachineOperand &O = ri.getOperand();
+    if (!O.isDef())
+      continue;
+
+    MachineInstr *MI = &*ri;
+    unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
+    if (!tii_->isMoveInstr(*MI, SrcReg, DstReg, SrcSubReg, DstSubReg))
+      return false;
+    if (SrcReg != DstInt.reg) {
+      OtherCopies.push_back(MI);
+      HaveConflict |= DstInt.liveAt(getInstructionIndex(MI));
+    } else {
+      IdentCopies.push_back(MI);
+      ++NumIdent;
+    }
+  }
+
+  if (!HaveConflict)
+    return false; // Let coalescer handle it
+  return IdentCopies.size() > OtherCopies.size();
+}
+
+void LiveIntervals::performEarlyCoalescing() {
+  if (!EarlyCoalescing)
+    return;
+
+  /// Perform early coalescing: eliminate copies which feed into phi joins
+  /// and whose sources are defined by the phi joins.
+  for (unsigned i = 0, e = phiJoinCopies.size(); i != e; ++i) {
+    MachineInstr *Join = phiJoinCopies[i];
+    if (CoalescingLimit != -1 && (int)numCoalescing == CoalescingLimit)
+      break;
+
+    unsigned PHISrc, PHIDst, SrcSubReg, DstSubReg;
+    bool isMove= tii_->isMoveInstr(*Join, PHISrc, PHIDst, SrcSubReg, DstSubReg);
+#ifndef NDEBUG
+    assert(isMove && "PHI join instruction must be a move!");
+#else
+    isMove = isMove;
+#endif
+
+    LiveInterval &DstInt = getInterval(PHIDst);
+    LiveInterval &SrcInt = getInterval(PHISrc);
+    SmallVector<MachineInstr*, 16> IdentCopies;
+    SmallVector<MachineInstr*, 16> OtherCopies;
+    if (!isProfitableToCoalesce(DstInt, SrcInt, IdentCopies, OtherCopies))
+      continue;
+
+    DEBUG(errs() << "PHI Join: " << *Join);
+    assert(DstInt.containsOneValue() && "PHI join should have just one val#!");
+    VNInfo *VNI = DstInt.getValNumInfo(0);
+
+    // Change the non-identity copies to directly target the phi destination.
+    for (unsigned i = 0, e = OtherCopies.size(); i != e; ++i) {
+      MachineInstr *PHICopy = OtherCopies[i];
+      DEBUG(errs() << "Moving: " << *PHICopy);
+
+      MachineInstrIndex MIIndex = getInstructionIndex(PHICopy);
+      MachineInstrIndex DefIndex = getDefIndex(MIIndex);
+      LiveRange *SLR = SrcInt.getLiveRangeContaining(DefIndex);
+      MachineInstrIndex StartIndex = SLR->start;
+      MachineInstrIndex EndIndex = SLR->end;
+
+      // Delete val# defined by the now identity copy and add the range from
+      // beginning of the mbb to the end of the range.
+      SrcInt.removeValNo(SLR->valno);
+      DEBUG(errs() << "  added range [" << StartIndex << ','
+            << EndIndex << "] to reg" << DstInt.reg << '\n');
+      if (DstInt.liveAt(StartIndex))
+        DstInt.removeRange(StartIndex, EndIndex);
+      VNInfo *NewVNI = DstInt.getNextValue(DefIndex, PHICopy, true,
+                                           VNInfoAllocator);
+      NewVNI->setHasPHIKill(true);
+      DstInt.addRange(LiveRange(StartIndex, EndIndex, NewVNI));
+      for (unsigned j = 0, ee = PHICopy->getNumOperands(); j != ee; ++j) {
+        MachineOperand &MO = PHICopy->getOperand(j);
+        if (!MO.isReg() || MO.getReg() != PHISrc)
+          continue;
+        MO.setReg(PHIDst);
+      }
+    }
+
+    // Now let's eliminate all the would-be identity copies.
+    for (unsigned i = 0, e = IdentCopies.size(); i != e; ++i) {
+      MachineInstr *PHICopy = IdentCopies[i];
+      DEBUG(errs() << "Coalescing: " << *PHICopy);
+
+      MachineInstrIndex MIIndex = getInstructionIndex(PHICopy);
+      MachineInstrIndex DefIndex = getDefIndex(MIIndex);
+      LiveRange *SLR = SrcInt.getLiveRangeContaining(DefIndex);
+      MachineInstrIndex StartIndex = SLR->start;
+      MachineInstrIndex EndIndex = SLR->end;
+
+      // Delete val# defined by the now identity copy and add the range from
+      // beginning of the mbb to the end of the range.
+      SrcInt.removeValNo(SLR->valno);
+      RemoveMachineInstrFromMaps(PHICopy);
+      PHICopy->eraseFromParent();
+      DEBUG(errs() << "  added range [" << StartIndex << ','
+            << EndIndex << "] to reg" << DstInt.reg << '\n');
+      DstInt.addRange(LiveRange(StartIndex, EndIndex, VNI));
+    }
+
+    // Remove the phi join and update the phi block liveness.
+    MachineInstrIndex MIIndex = getInstructionIndex(Join);
+    MachineInstrIndex UseIndex = getUseIndex(MIIndex);
+    MachineInstrIndex DefIndex = getDefIndex(MIIndex);
+    LiveRange *SLR = SrcInt.getLiveRangeContaining(UseIndex);
+    LiveRange *DLR = DstInt.getLiveRangeContaining(DefIndex);
+    DLR->valno->setCopy(0);
+    DLR->valno->setIsDefAccurate(false);
+    DstInt.addRange(LiveRange(SLR->start, SLR->end, DLR->valno));
+    SrcInt.removeRange(SLR->start, SLR->end);
+    assert(SrcInt.empty());
+    removeInterval(PHISrc);
+    RemoveMachineInstrFromMaps(Join);
+    Join->eraseFromParent();
+
+    ++numCoalescing;
+  }
  }
  
  /// computeIntervals - computes the live intervals for virtual
  }
  
  /// computeIntervals - computes the live intervals for virtual
@@ -740,17 +1215,17 @@ exit:
  /// live interval is an interval [i, j) where 1 <= i <= j < N for
  /// which a variable is live
  void LiveIntervals::computeIntervals() { 
  /// live interval is an interval [i, j) where 1 <= i <= j < N for
  /// which a variable is live
  void LiveIntervals::computeIntervals() { 
+  DEBUG(errs() << "********** COMPUTING LIVE INTERVALS **********\n"
+               << "********** Function: "
+               << ((Value*)mf_->getFunction())->getName() << '\n');
  
  
-  DOUT << "********** COMPUTING LIVE INTERVALS **********\n"
-       << "********** Function: "
-       << ((Value*)mf_->getFunction())->getName() << '\n';
-  
+  SmallVector<unsigned, 8> UndefUses;
    for (MachineFunction::iterator MBBI = mf_->begin(), E = mf_->end();
         MBBI != E; ++MBBI) {
      MachineBasicBlock *MBB = MBBI;
      // Track the index of the current machine instr.
    for (MachineFunction::iterator MBBI = mf_->begin(), E = mf_->end();
         MBBI != E; ++MBBI) {
      MachineBasicBlock *MBB = MBBI;
      // Track the index of the current machine instr.
-    unsigned MIIndex = getMBBStartIdx(MBB);
-    DOUT << ((Value*)MBB->getBasicBlock())->getName() << ":\n";
+    MachineInstrIndex MIIndex = getMBBStartIdx(MBB);
+    DEBUG(errs() << ((Value*)MBB->getBasicBlock())->getName() << ":\n");
  
      MachineBasicBlock::iterator MI = MBB->begin(), miEnd = MBB->end();
  
  
      MachineBasicBlock::iterator MI = MBB->begin(), miEnd = MBB->end();
  
@@ -766,37 +1241,52 @@ void LiveIntervals::computeIntervals() {
      }
      
      // Skip over empty initial indices.
      }
      
      // Skip over empty initial indices.
-    while (MIIndex / InstrSlots::NUM < i2miMap_.size() &&
+    while (MIIndex.getVecIndex() < i2miMap_.size() &&
             getInstructionFromIndex(MIIndex) == 0)
             getInstructionFromIndex(MIIndex) == 0)
-      MIIndex += InstrSlots::NUM;
+      MIIndex = getNextIndex(MIIndex);
      
      for (; MI != miEnd; ++MI) {
      
      for (; MI != miEnd; ++MI) {
-      DOUT << MIIndex << "\t" << *MI;
+      DEBUG(errs() << MIIndex << "\t" << *MI);
  
        // Handle defs.
        for (int i = MI->getNumOperands() - 1; i >= 0; --i) {
          MachineOperand &MO = MI->getOperand(i);
  
        // Handle defs.
        for (int i = MI->getNumOperands() - 1; i >= 0; --i) {
          MachineOperand &MO = MI->getOperand(i);
+        if (!MO.isReg() || !MO.getReg())
+          continue;
+
          // handle register defs - build intervals
          // handle register defs - build intervals
-        if (MO.isReg() && MO.getReg() && MO.isDef()) {
+        if (MO.isDef())
            handleRegisterDef(MBB, MI, MIIndex, MO, i);
            handleRegisterDef(MBB, MI, MIIndex, MO, i);
-        }
+        else if (MO.isUndef())
+          UndefUses.push_back(MO.getReg());
        }
  
        // Skip over the empty slots after each instruction.
        unsigned Slots = MI->getDesc().getNumDefs();
        if (Slots == 0)
          Slots = 1;
        }
  
        // Skip over the empty slots after each instruction.
        unsigned Slots = MI->getDesc().getNumDefs();
        if (Slots == 0)
          Slots = 1;
-      MIIndex += InstrSlots::NUM * Slots;
+
+      while (Slots--)
+        MIIndex = getNextIndex(MIIndex);
        
        // Skip over empty indices.
        
        // Skip over empty indices.
-      while (MIIndex / InstrSlots::NUM < i2miMap_.size() &&
+      while (MIIndex.getVecIndex() < i2miMap_.size() &&
               getInstructionFromIndex(MIIndex) == 0)
               getInstructionFromIndex(MIIndex) == 0)
-        MIIndex += InstrSlots::NUM;
+        MIIndex = getNextIndex(MIIndex);
      }
    }
      }
    }
+
+  // Create empty intervals for registers defined by implicit_def's (except
+  // for those implicit_def that define values which are liveout of their
+  // blocks.
+  for (unsigned i = 0, e = UndefUses.size(); i != e; ++i) {
+    unsigned UndefReg = UndefUses[i];
+    (void)getOrCreateInterval(UndefReg);
+  }
  }
  
  }
  
-bool LiveIntervals::findLiveInMBBs(unsigned Start, unsigned End,
+bool LiveIntervals::findLiveInMBBs(
+                              MachineInstrIndex Start, MachineInstrIndex End,
                                SmallVectorImpl<MachineBasicBlock*> &MBBs) const {
    std::vector<IdxMBBPair>::const_iterator I =
      std::lower_bound(Idx2MBBMap.begin(), Idx2MBBMap.end(), Start);
                                SmallVectorImpl<MachineBasicBlock*> &MBBs) const {
    std::vector<IdxMBBPair>::const_iterator I =
      std::lower_bound(Idx2MBBMap.begin(), Idx2MBBMap.end(), Start);
@@ -812,7 +1302,8 @@ bool LiveIntervals::findLiveInMBBs(unsigned Start, unsigned End,
    return ResVal;
  }
  
    return ResVal;
  }
  
-bool LiveIntervals::findReachableMBBs(unsigned Start, unsigned End,
+bool LiveIntervals::findReachableMBBs(
+                              MachineInstrIndex Start, MachineInstrIndex End,
                                SmallVectorImpl<MachineBasicBlock*> &MBBs) const {
    std::vector<IdxMBBPair>::const_iterator I =
      std::lower_bound(Idx2MBBMap.begin(), Idx2MBBMap.end(), Start);
                                SmallVectorImpl<MachineBasicBlock*> &MBBs) const {
    std::vector<IdxMBBPair>::const_iterator I =
      std::lower_bound(Idx2MBBMap.begin(), Idx2MBBMap.end(), Start);
@@ -842,30 +1333,30 @@ LiveInterval* LiveIntervals::createInterval(unsigned reg) {
  /// managing the allocated memory.
  LiveInterval* LiveIntervals::dupInterval(LiveInterval *li) {
    LiveInterval *NewLI = createInterval(li->reg);
  /// managing the allocated memory.
  LiveInterval* LiveIntervals::dupInterval(LiveInterval *li) {
    LiveInterval *NewLI = createInterval(li->reg);
-  NewLI->Copy(*li, getVNInfoAllocator());
+  NewLI->Copy(*li, mri_, getVNInfoAllocator());
    return NewLI;
  }
  
  /// getVNInfoSourceReg - Helper function that parses the specified VNInfo
  /// copy field and returns the source register that defines it.
  unsigned LiveIntervals::getVNInfoSourceReg(const VNInfo *VNI) const {
    return NewLI;
  }
  
  /// getVNInfoSourceReg - Helper function that parses the specified VNInfo
  /// copy field and returns the source register that defines it.
  unsigned LiveIntervals::getVNInfoSourceReg(const VNInfo *VNI) const {
-  if (!VNI->copy)
+  if (!VNI->getCopy())
      return 0;
  
      return 0;
  
-  if (VNI->copy->getOpcode() == TargetInstrInfo::EXTRACT_SUBREG) {
+  if (VNI->getCopy()->getOpcode() == TargetInstrInfo::EXTRACT_SUBREG) {
      // If it's extracting out of a physical register, return the sub-register.
      // If it's extracting out of a physical register, return the sub-register.
-    unsigned Reg = VNI->copy->getOperand(1).getReg();
+    unsigned Reg = VNI->getCopy()->getOperand(1).getReg();
      if (TargetRegisterInfo::isPhysicalRegister(Reg))
      if (TargetRegisterInfo::isPhysicalRegister(Reg))
-      Reg = tri_->getSubReg(Reg, VNI->copy->getOperand(2).getImm());
+      Reg = tri_->getSubReg(Reg, VNI->getCopy()->getOperand(2).getImm());
      return Reg;
      return Reg;
-  } else if (VNI->copy->getOpcode() == TargetInstrInfo::INSERT_SUBREG ||
-             VNI->copy->getOpcode() == TargetInstrInfo::SUBREG_TO_REG)
-    return VNI->copy->getOperand(2).getReg();
+  } else if (VNI->getCopy()->getOpcode() == TargetInstrInfo::INSERT_SUBREG ||
+             VNI->getCopy()->getOpcode() == TargetInstrInfo::SUBREG_TO_REG)
+    return VNI->getCopy()->getOperand(2).getReg();
  
    unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
  
    unsigned SrcReg, DstReg, SrcSubReg, DstSubReg;
-  if (tii_->isMoveInstr(*VNI->copy, SrcReg, DstReg, SrcSubReg, DstSubReg))
+  if (tii_->isMoveInstr(*VNI->getCopy(), SrcReg, DstReg, SrcSubReg, DstSubReg))
      return SrcReg;
      return SrcReg;
-  assert(0 && "Unrecognized copy instruction!");
+  llvm_unreachable("Unrecognized copy instruction!");
    return 0;
  }
  
    return 0;
  }
  
@@ -886,6 +1377,10 @@ unsigned LiveIntervals::getReMatImplicitUse(const LiveInterval &li,
      unsigned Reg = MO.getReg();
      if (Reg == 0 || Reg == li.reg)
        continue;
      unsigned Reg = MO.getReg();
      if (Reg == 0 || Reg == li.reg)
        continue;
+    
+    if (TargetRegisterInfo::isPhysicalRegister(Reg) &&
+        !allocatableRegs_[Reg])
+      continue;
      // FIXME: For now, only remat MI with at most one register operand.
      assert(!RegOp &&
             "Can't rematerialize instruction with multiple register operand!");
      // FIXME: For now, only remat MI with at most one register operand.
      assert(!RegOp &&
             "Can't rematerialize instruction with multiple register operand!");
@@ -900,8 +1395,8 @@ unsigned LiveIntervals::getReMatImplicitUse(const LiveInterval &li,
  /// isValNoAvailableAt - Return true if the val# of the specified interval
  /// which reaches the given instruction also reaches the specified use index.
  bool LiveIntervals::isValNoAvailableAt(const LiveInterval &li, MachineInstr *MI,
  /// isValNoAvailableAt - Return true if the val# of the specified interval
  /// which reaches the given instruction also reaches the specified use index.
  bool LiveIntervals::isValNoAvailableAt(const LiveInterval &li, MachineInstr *MI,
-                                       unsigned UseIdx) const {
-  unsigned Index = getInstructionIndex(MI);  
+                                       MachineInstrIndex UseIdx) const {
+  MachineInstrIndex Index = getInstructionIndex(MI);  
    VNInfo *ValNo = li.FindLiveRangeContaining(Index)->valno;
    LiveInterval::const_iterator UI = li.FindLiveRangeContaining(UseIdx);
    return UI != li.end() && UI->valno == ValNo;
    VNInfo *ValNo = li.FindLiveRangeContaining(Index)->valno;
    LiveInterval::const_iterator UI = li.FindLiveRangeContaining(UseIdx);
    return UI != li.end() && UI->valno == ValNo;
@@ -1011,7 +1506,7 @@ bool LiveIntervals::isReMaterializable(const LiveInterval &li,
      for (MachineRegisterInfo::use_iterator ri = mri_->use_begin(li.reg),
             re = mri_->use_end(); ri != re; ++ri) {
        MachineInstr *UseMI = &*ri;
      for (MachineRegisterInfo::use_iterator ri = mri_->use_begin(li.reg),
             re = mri_->use_end(); ri != re; ++ri) {
        MachineInstr *UseMI = &*ri;
-      unsigned UseIdx = getInstructionIndex(UseMI);
+      MachineInstrIndex UseIdx = getInstructionIndex(UseMI);
        if (li.FindLiveRangeContaining(UseIdx)->valno != ValNo)
          continue;
        if (!isValNoAvailableAt(ImpLi, MI, UseIdx))
        if (li.FindLiveRangeContaining(UseIdx)->valno != ValNo)
          continue;
        if (!isValNoAvailableAt(ImpLi, MI, UseIdx))
@@ -1045,13 +1540,12 @@ bool LiveIntervals::isReMaterializable(const LiveInterval &li,
    for (LiveInterval::const_vni_iterator i = li.vni_begin(), e = li.vni_end();
         i != e; ++i) {
      const VNInfo *VNI = *i;
    for (LiveInterval::const_vni_iterator i = li.vni_begin(), e = li.vni_end();
         i != e; ++i) {
      const VNInfo *VNI = *i;
-    unsigned DefIdx = VNI->def;
-    if (DefIdx == ~1U)
+    if (VNI->isUnused())
        continue; // Dead val#.
      // Is the def for the val# rematerializable?
        continue; // Dead val#.
      // Is the def for the val# rematerializable?
-    if (DefIdx == ~0u)
+    if (!VNI->isDefAccurate())
        return false;
        return false;
-    MachineInstr *ReMatDefMI = getInstructionFromIndex(DefIdx);
+    MachineInstr *ReMatDefMI = getInstructionFromIndex(VNI->def);
      bool DefIsLoad = false;
      if (!ReMatDefMI ||
          !isReMaterializable(li, VNI, ReMatDefMI, SpillIs, DefIsLoad))
      bool DefIsLoad = false;
      if (!ReMatDefMI ||
          !isReMaterializable(li, VNI, ReMatDefMI, SpillIs, DefIsLoad))
@@ -1097,7 +1591,7 @@ static bool FilterFoldedOps(MachineInstr *MI,
  /// returns true.
  bool LiveIntervals::tryFoldMemoryOperand(MachineInstr* &MI,
                                           VirtRegMap &vrm, MachineInstr *DefMI,
  /// returns true.
  bool LiveIntervals::tryFoldMemoryOperand(MachineInstr* &MI,
                                           VirtRegMap &vrm, MachineInstr *DefMI,
-                                         unsigned InstrIdx,
+                                         MachineInstrIndex InstrIdx,
                                           SmallVector<unsigned, 2> &Ops,
                                           bool isSS, int Slot, unsigned Reg) {
    // If it is an implicit def instruction, just delete it.
                                           SmallVector<unsigned, 2> &Ops,
                                           bool isSS, int Slot, unsigned Reg) {
    // If it is an implicit def instruction, just delete it.
@@ -1136,7 +1630,7 @@ bool LiveIntervals::tryFoldMemoryOperand(MachineInstr* &MI,
      vrm.transferRestorePts(MI, fmi);
      vrm.transferEmergencySpills(MI, fmi);
      mi2iMap_.erase(MI);
      vrm.transferRestorePts(MI, fmi);
      vrm.transferEmergencySpills(MI, fmi);
      mi2iMap_.erase(MI);
-    i2miMap_[InstrIdx /InstrSlots::NUM] = fmi;
+    i2miMap_[InstrIdx.getVecIndex()] = fmi;
      mi2iMap_[fmi] = InstrIdx;
      MI = MBB.insert(MBB.erase(MI), fmi);
      ++numFolds;
      mi2iMap_[fmi] = InstrIdx;
      MI = MBB.insert(MBB.erase(MI), fmi);
      ++numFolds;
@@ -1209,7 +1703,8 @@ void LiveIntervals::rewriteImplicitOps(const LiveInterval &li,
  /// for addIntervalsForSpills to rewrite uses / defs for the given live range.
  bool LiveIntervals::
  rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
  /// for addIntervalsForSpills to rewrite uses / defs for the given live range.
  bool LiveIntervals::
  rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
-                 bool TrySplit, unsigned index, unsigned end,  MachineInstr *MI,
+                 bool TrySplit, MachineInstrIndex index, MachineInstrIndex end, 
+                 MachineInstr *MI,
                   MachineInstr *ReMatOrigDefMI, MachineInstr *ReMatDefMI,
                   unsigned Slot, int LdSlot,
                   bool isLoad, bool isLoadSS, bool DefIsReMat, bool CanDelete,
                   MachineInstr *ReMatOrigDefMI, MachineInstr *ReMatDefMI,
                   unsigned Slot, int LdSlot,
                   bool isLoad, bool isLoadSS, bool DefIsReMat, bool CanDelete,
@@ -1219,9 +1714,7 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
                   const MachineLoopInfo *loopInfo,
                   unsigned &NewVReg, unsigned ImpUse, bool &HasDef, bool &HasUse,
                   DenseMap<unsigned,unsigned> &MBBVRegsMap,
                   const MachineLoopInfo *loopInfo,
                   unsigned &NewVReg, unsigned ImpUse, bool &HasDef, bool &HasUse,
                   DenseMap<unsigned,unsigned> &MBBVRegsMap,
-                 std::vector<LiveInterval*> &NewLIs, float &SSWeight) {
-  MachineBasicBlock *MBB = MI->getParent();
-  unsigned loopDepth = loopInfo->getLoopDepth(MBB);
+                 std::vector<LiveInterval*> &NewLIs) {
    bool CanFold = false;
   RestartInstruction:
    for (unsigned i = 0; i != MI->getNumOperands(); ++i) {
    bool CanFold = false;
   RestartInstruction:
    for (unsigned i = 0; i != MI->getNumOperands(); ++i) {
@@ -1242,8 +1735,8 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
        // If this is the rematerializable definition MI itself and
        // all of its uses are rematerialized, simply delete it.
        if (MI == ReMatOrigDefMI && CanDelete) {
        // If this is the rematerializable definition MI itself and
        // all of its uses are rematerialized, simply delete it.
        if (MI == ReMatOrigDefMI && CanDelete) {
-        DOUT << "\t\t\t\tErasing re-materlizable def: ";
-        DOUT << MI << '\n';
+        DEBUG(errs() << "\t\t\t\tErasing re-materlizable def: "
+                     << MI << '\n');
          RemoveMachineInstrFromMaps(MI);
          vrm.RemoveMachineInstrFromMaps(MI);
          MI->eraseFromParent();
          RemoveMachineInstrFromMaps(MI);
          vrm.RemoveMachineInstrFromMaps(MI);
          MI->eraseFromParent();
@@ -1285,28 +1778,13 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
          continue;
        if (RegJ == RegI) {
          Ops.push_back(j);
          continue;
        if (RegJ == RegI) {
          Ops.push_back(j);
-        HasUse |= MOj.isUse();
-        HasDef |= MOj.isDef();
+        if (!MOj.isUndef()) {
+          HasUse |= MOj.isUse();
+          HasDef |= MOj.isDef();
+        }
        }
      }
  
        }
      }
  
-    if (HasUse && !li.liveAt(getUseIndex(index)))
-      // Must be defined by an implicit def. It should not be spilled. Note,
-      // this is for correctness reason. e.g.
-      // 8   %reg1024<def> = IMPLICIT_DEF
-      // 12  %reg1024<def> = INSERT_SUBREG %reg1024<kill>, %reg1025, 2
-      // The live range [12, 14) are not part of the r1024 live interval since
-      // it's defined by an implicit def. It will not conflicts with live
-      // interval of r1025. Now suppose both registers are spilled, you can
-      // easily see a situation where both registers are reloaded before
-      // the INSERT_SUBREG and both target registers that would overlap.
-      HasUse = false;
-
-    // Update stack slot spill weight if we are splitting.
-    float Weight = getSpillWeight(HasDef, HasUse, loopDepth);
-      if (!TrySplit)
-      SSWeight += Weight;
-
      // Create a new virtual register for the spill interval.
      // Create the new register now so we can map the fold instruction
      // to the new register so when it is unfolded we get the correct
      // Create a new virtual register for the spill interval.
      // Create the new register now so we can map the fold instruction
      // to the new register so when it is unfolded we get the correct
@@ -1338,10 +1816,8 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
            HasUse = false;
            HasDef = false;
            CanFold = false;
            HasUse = false;
            HasDef = false;
            CanFold = false;
-          if (isRemoved(MI)) {
-            SSWeight -= Weight;
+          if (isNotInMIMap(MI))
              break;
              break;
-          }
            goto RestartInstruction;
          }
        } else {
            goto RestartInstruction;
          }
        } else {
@@ -1364,7 +1840,7 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
              
      if (CreatedNewVReg) {
        if (DefIsReMat) {
              
      if (CreatedNewVReg) {
        if (DefIsReMat) {
-        vrm.setVirtIsReMaterialized(NewVReg, ReMatDefMI/*, CanDelete*/);
+        vrm.setVirtIsReMaterialized(NewVReg, ReMatDefMI);
          if (ReMatIds[VNI->id] == VirtRegMap::MAX_STACK_SLOT) {
            // Each valnum may have its own remat id.
            ReMatIds[VNI->id] = vrm.assignVirtReMatId(NewVReg);
          if (ReMatIds[VNI->id] == VirtRegMap::MAX_STACK_SLOT) {
            // Each valnum may have its own remat id.
            ReMatIds[VNI->id] = vrm.assignVirtReMatId(NewVReg);
@@ -1393,7 +1869,7 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
      if (DefIsReMat && ImpUse)
        MI->addOperand(MachineOperand::CreateReg(ImpUse, false, true));
  
      if (DefIsReMat && ImpUse)
        MI->addOperand(MachineOperand::CreateReg(ImpUse, false, true));
  
-    // create a new register interval for this spill / remat.
+    // Create a new register interval for this spill / remat.
      LiveInterval &nI = getOrCreateInterval(NewVReg);
      if (CreatedNewVReg) {
        NewLIs.push_back(&nI);
      LiveInterval &nI = getOrCreateInterval(NewVReg);
      if (CreatedNewVReg) {
        NewLIs.push_back(&nI);
@@ -1404,38 +1880,46 @@ rewriteInstructionForSpills(const LiveInterval &li, const VNInfo *VNI,
  
      if (HasUse) {
        if (CreatedNewVReg) {
  
      if (HasUse) {
        if (CreatedNewVReg) {
-        LiveRange LR(getLoadIndex(index), getUseIndex(index)+1,
-                     nI.getNextValue(~0U, 0, VNInfoAllocator));
-        DOUT << " +" << LR;
+        LiveRange LR(getLoadIndex(index), getNextSlot(getUseIndex(index)),
+                     nI.getNextValue(MachineInstrIndex(), 0, false,
+                                     VNInfoAllocator));
+        DEBUG(errs() << " +" << LR);
          nI.addRange(LR);
        } else {
          // Extend the split live interval to this def / use.
          nI.addRange(LR);
        } else {
          // Extend the split live interval to this def / use.
-        unsigned End = getUseIndex(index)+1;
+        MachineInstrIndex End = getNextSlot(getUseIndex(index));
          LiveRange LR(nI.ranges[nI.ranges.size()-1].end, End,
                       nI.getValNumInfo(nI.getNumValNums()-1));
          LiveRange LR(nI.ranges[nI.ranges.size()-1].end, End,
                       nI.getValNumInfo(nI.getNumValNums()-1));
-        DOUT << " +" << LR;
+        DEBUG(errs() << " +" << LR);
          nI.addRange(LR);
        }
      }
      if (HasDef) {
        LiveRange LR(getDefIndex(index), getStoreIndex(index),
          nI.addRange(LR);
        }
      }
      if (HasDef) {
        LiveRange LR(getDefIndex(index), getStoreIndex(index),
-                   nI.getNextValue(~0U, 0, VNInfoAllocator));
-      DOUT << " +" << LR;
+                   nI.getNextValue(MachineInstrIndex(), 0, false,
+                                   VNInfoAllocator));
+      DEBUG(errs() << " +" << LR);
        nI.addRange(LR);
      }
  
        nI.addRange(LR);
      }
  
-    DOUT << "\t\t\t\tAdded new interval: ";
-    nI.print(DOUT, tri_);
-    DOUT << '\n';
+    DEBUG({
+        errs() << "\t\t\t\tAdded new interval: ";
+        nI.print(errs(), tri_);
+        errs() << '\n';
+      });
    }
    return CanFold;
  }
  bool LiveIntervals::anyKillInMBBAfterIdx(const LiveInterval &li,
                                     const VNInfo *VNI,
    }
    return CanFold;
  }
  bool LiveIntervals::anyKillInMBBAfterIdx(const LiveInterval &li,
                                     const VNInfo *VNI,
-                                   MachineBasicBlock *MBB, unsigned Idx) const {
-  unsigned End = getMBBEndIdx(MBB);
+                                   MachineBasicBlock *MBB,
+                                   MachineInstrIndex Idx) const {
+  MachineInstrIndex End = getMBBEndIdx(MBB);
    for (unsigned j = 0, ee = VNI->kills.size(); j != ee; ++j) {
    for (unsigned j = 0, ee = VNI->kills.size(); j != ee; ++j) {
-    unsigned KillIdx = VNI->kills[j];
+    if (VNI->kills[j].isPHIIndex())
+      continue;
+
+    MachineInstrIndex KillIdx = VNI->kills[j];
      if (KillIdx > Idx && KillIdx < End)
        return true;
    }
      if (KillIdx > Idx && KillIdx < End)
        return true;
    }
@@ -1446,11 +1930,11 @@ bool LiveIntervals::anyKillInMBBAfterIdx(const LiveInterval &li,
  /// during spilling.
  namespace {
    struct RewriteInfo {
  /// during spilling.
  namespace {
    struct RewriteInfo {
-    unsigned Index;
+    MachineInstrIndex Index;
      MachineInstr *MI;
      bool HasUse;
      bool HasDef;
      MachineInstr *MI;
      bool HasUse;
      bool HasDef;
-    RewriteInfo(unsigned i, MachineInstr *mi, bool u, bool d)
+    RewriteInfo(MachineInstrIndex i, MachineInstr *mi, bool u, bool d)
        : Index(i), MI(mi), HasUse(u), HasDef(d) {}
    };
  
        : Index(i), MI(mi), HasUse(u), HasDef(d) {}
    };
  
@@ -1476,11 +1960,11 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
                      BitVector &RestoreMBBs,
                      DenseMap<unsigned, std::vector<SRInfo> > &RestoreIdxes,
                      DenseMap<unsigned,unsigned> &MBBVRegsMap,
                      BitVector &RestoreMBBs,
                      DenseMap<unsigned, std::vector<SRInfo> > &RestoreIdxes,
                      DenseMap<unsigned,unsigned> &MBBVRegsMap,
-                    std::vector<LiveInterval*> &NewLIs, float &SSWeight) {
+                    std::vector<LiveInterval*> &NewLIs) {
    bool AllCanFold = true;
    unsigned NewVReg = 0;
    bool AllCanFold = true;
    unsigned NewVReg = 0;
-  unsigned start = getBaseIndex(I->start);
-  unsigned end = getBaseIndex(I->end-1) + InstrSlots::NUM;
+  MachineInstrIndex start = getBaseIndex(I->start);
+  MachineInstrIndex end = getNextIndex(getBaseIndex(getPrevSlot(I->end)));
  
    // First collect all the def / use in this live range that will be rewritten.
    // Make sure they are sorted according to instruction index.
  
    // First collect all the def / use in this live range that will be rewritten.
    // Make sure they are sorted according to instruction index.
@@ -1491,10 +1975,11 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
      MachineOperand &O = ri.getOperand();
      ++ri;
      assert(!O.isImplicit() && "Spilling register that's used as implicit use?");
      MachineOperand &O = ri.getOperand();
      ++ri;
      assert(!O.isImplicit() && "Spilling register that's used as implicit use?");
-    unsigned index = getInstructionIndex(MI);
+    MachineInstrIndex index = getInstructionIndex(MI);
      if (index < start || index >= end)
        continue;
      if (index < start || index >= end)
        continue;
-    if (O.isUse() && !li.liveAt(getUseIndex(index)))
+
+    if (O.isUndef())
        // Must be defined by an implicit def. It should not be spilled. Note,
        // this is for correctness reason. e.g.
        // 8   %reg1024<def> = IMPLICIT_DEF
        // Must be defined by an implicit def. It should not be spilled. Note,
        // this is for correctness reason. e.g.
        // 8   %reg1024<def> = IMPLICIT_DEF
@@ -1514,7 +1999,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
    for (unsigned i = 0, e = RewriteMIs.size(); i != e; ) {
      RewriteInfo &rwi = RewriteMIs[i];
      ++i;
    for (unsigned i = 0, e = RewriteMIs.size(); i != e; ) {
      RewriteInfo &rwi = RewriteMIs[i];
      ++i;
-    unsigned index = rwi.Index;
+    MachineInstrIndex index = rwi.Index;
      bool MIHasUse = rwi.HasUse;
      bool MIHasDef = rwi.HasDef;
      MachineInstr *MI = rwi.MI;
      bool MIHasUse = rwi.HasUse;
      bool MIHasDef = rwi.HasDef;
      MachineInstr *MI = rwi.MI;
@@ -1578,7 +2063,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
                           index, end, MI, ReMatOrigDefMI, ReMatDefMI,
                           Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                           CanDelete, vrm, rc, ReMatIds, loopInfo, NewVReg,
                           index, end, MI, ReMatOrigDefMI, ReMatDefMI,
                           Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                           CanDelete, vrm, rc, ReMatIds, loopInfo, NewVReg,
-                         ImpUse, HasDef, HasUse, MBBVRegsMap, NewLIs, SSWeight);
+                         ImpUse, HasDef, HasUse, MBBVRegsMap, NewLIs);
      if (!HasDef && !HasUse)
        continue;
  
      if (!HasDef && !HasUse)
        continue;
  
@@ -1600,7 +2085,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
            HasKill = anyKillInMBBAfterIdx(li, I->valno, MBB, getDefIndex(index));
          else {
            // If this is a two-address code, then this index starts a new VNInfo.
            HasKill = anyKillInMBBAfterIdx(li, I->valno, MBB, getDefIndex(index));
          else {
            // If this is a two-address code, then this index starts a new VNInfo.
-          const VNInfo *VNI = li.findDefinedVNInfo(getDefIndex(index));
+          const VNInfo *VNI = li.findDefinedVNInfoForRegInt(getDefIndex(index));
            if (VNI)
              HasKill = anyKillInMBBAfterIdx(li, VNI, MBB, getDefIndex(index));
          }
            if (VNI)
              HasKill = anyKillInMBBAfterIdx(li, VNI, MBB, getDefIndex(index));
          }
@@ -1613,7 +2098,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
              SpillIdxes.insert(std::make_pair(MBBId, S));
            } else if (SII->second.back().vreg != NewVReg) {
              SII->second.push_back(SRInfo(index, NewVReg, true));
              SpillIdxes.insert(std::make_pair(MBBId, S));
            } else if (SII->second.back().vreg != NewVReg) {
              SII->second.push_back(SRInfo(index, NewVReg, true));
-          } else if ((int)index > SII->second.back().index) {
+          } else if (index > SII->second.back().index) {
              // If there is an earlier def and this is a two-address
              // instruction, then it's not possible to fold the store (which
              // would also fold the load).
              // If there is an earlier def and this is a two-address
              // instruction, then it's not possible to fold the store (which
              // would also fold the load).
@@ -1624,7 +2109,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
            SpillMBBs.set(MBBId);
          } else if (SII != SpillIdxes.end() &&
                     SII->second.back().vreg == NewVReg &&
            SpillMBBs.set(MBBId);
          } else if (SII != SpillIdxes.end() &&
                     SII->second.back().vreg == NewVReg &&
-                   (int)index > SII->second.back().index) {
+                   index > SII->second.back().index) {
            // There is an earlier def that's not killed (must be two-address).
            // The spill is no longer needed.
            SII->second.pop_back();
            // There is an earlier def that's not killed (must be two-address).
            // The spill is no longer needed.
            SII->second.pop_back();
@@ -1641,7 +2126,7 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
          SpillIdxes.find(MBBId);
        if (SII != SpillIdxes.end() &&
            SII->second.back().vreg == NewVReg &&
          SpillIdxes.find(MBBId);
        if (SII != SpillIdxes.end() &&
            SII->second.back().vreg == NewVReg &&
-          (int)index > SII->second.back().index)
+          index > SII->second.back().index)
          // Use(s) following the last def, it's not safe to fold the spill.
          SII->second.back().canFold = false;
        DenseMap<unsigned, std::vector<SRInfo> >::iterator RII =
          // Use(s) following the last def, it's not safe to fold the spill.
          SII->second.back().canFold = false;
        DenseMap<unsigned, std::vector<SRInfo> >::iterator RII =
@@ -1675,8 +2160,8 @@ rewriteInstructionsForSpills(const LiveInterval &li, bool TrySplit,
    }
  }
  
    }
  }
  
-bool LiveIntervals::alsoFoldARestore(int Id, int index, unsigned vr,
-                        BitVector &RestoreMBBs,
+bool LiveIntervals::alsoFoldARestore(int Id, MachineInstrIndex index,
+                        unsigned vr, BitVector &RestoreMBBs,
                          DenseMap<unsigned,std::vector<SRInfo> > &RestoreIdxes) {
    if (!RestoreMBBs[Id])
      return false;
                          DenseMap<unsigned,std::vector<SRInfo> > &RestoreIdxes) {
    if (!RestoreMBBs[Id])
      return false;
@@ -1689,15 +2174,15 @@ bool LiveIntervals::alsoFoldARestore(int Id, int index, unsigned vr,
    return false;
  }
  
    return false;
  }
  
-void LiveIntervals::eraseRestoreInfo(int Id, int index, unsigned vr,
-                        BitVector &RestoreMBBs,
+void LiveIntervals::eraseRestoreInfo(int Id, MachineInstrIndex index,
+                        unsigned vr, BitVector &RestoreMBBs,
                          DenseMap<unsigned,std::vector<SRInfo> > &RestoreIdxes) {
    if (!RestoreMBBs[Id])
      return;
    std::vector<SRInfo> &Restores = RestoreIdxes[Id];
    for (unsigned i = 0, e = Restores.size(); i != e; ++i)
      if (Restores[i].index == index && Restores[i].vreg)
                          DenseMap<unsigned,std::vector<SRInfo> > &RestoreIdxes) {
    if (!RestoreMBBs[Id])
      return;
    std::vector<SRInfo> &Restores = RestoreIdxes[Id];
    for (unsigned i = 0, e = Restores.size(); i != e; ++i)
      if (Restores[i].index == index && Restores[i].vreg)
-      Restores[i].index = -1;
+      Restores[i].index = MachineInstrIndex();
  }
  
  /// handleSpilledImpDefs - Remove IMPLICIT_DEF instructions which are being
  }
  
  /// handleSpilledImpDefs - Remove IMPLICIT_DEF instructions which are being
@@ -1727,25 +2212,19 @@ LiveIntervals::handleSpilledImpDefs(const LiveInterval &li, VirtRegMap &vrm,
        NewLIs.push_back(&getOrCreateInterval(NewVReg));
        for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
          MachineOperand &MO = MI->getOperand(i);
        NewLIs.push_back(&getOrCreateInterval(NewVReg));
        for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
          MachineOperand &MO = MI->getOperand(i);
-        if (MO.isReg() && MO.getReg() == li.reg)
+        if (MO.isReg() && MO.getReg() == li.reg) {
            MO.setReg(NewVReg);
            MO.setReg(NewVReg);
+          MO.setIsUndef();
+        }
        }
      }
    }
  }
  
        }
      }
    }
  }
  
-namespace {
-  struct LISorter {
-    bool operator()(LiveInterval* A, LiveInterval* B) {
-      return A->beginNumber() < B->beginNumber();
-    }
-  };
-}
-
  std::vector<LiveInterval*> LiveIntervals::
  addIntervalsForSpillsFast(const LiveInterval &li,
                            const MachineLoopInfo *loopInfo,
  std::vector<LiveInterval*> LiveIntervals::
  addIntervalsForSpillsFast(const LiveInterval &li,
                            const MachineLoopInfo *loopInfo,
-                          VirtRegMap &vrm, float& SSWeight) {
+                          VirtRegMap &vrm) {
    unsigned slot = vrm.assignVirt2StackSlot(li.reg);
  
    std::vector<LiveInterval*> added;
    unsigned slot = vrm.assignVirt2StackSlot(li.reg);
  
    std::vector<LiveInterval*> added;
@@ -1753,14 +2232,14 @@ addIntervalsForSpillsFast(const LiveInterval &li,
    assert(li.weight != HUGE_VALF &&
           "attempt to spill already spilled interval!");
  
    assert(li.weight != HUGE_VALF &&
           "attempt to spill already spilled interval!");
  
-  DOUT << "\t\t\t\tadding intervals for spills for interval: ";
-  DEBUG(li.dump());
-  DOUT << '\n';
+  DEBUG({
+      errs() << "\t\t\t\tadding intervals for spills for interval: ";
+      li.dump();
+      errs() << '\n';
+    });
  
    const TargetRegisterClass* rc = mri_->getRegClass(li.reg);
  
  
    const TargetRegisterClass* rc = mri_->getRegClass(li.reg);
  
-  SSWeight = 0.0f;
-
    MachineRegisterInfo::reg_iterator RI = mri_->reg_begin(li.reg);
    while (RI != mri_->reg_end()) {
      MachineInstr* MI = &*RI;
    MachineRegisterInfo::reg_iterator RI = mri_->reg_begin(li.reg);
    while (RI != mri_->reg_end()) {
      MachineInstr* MI = &*RI;
@@ -1802,66 +2281,56 @@ addIntervalsForSpillsFast(const LiveInterval &li,
        }
        
        // Fill in  the new live interval.
        }
        
        // Fill in  the new live interval.
-      unsigned index = getInstructionIndex(MI);
+      MachineInstrIndex index = getInstructionIndex(MI);
        if (HasUse) {
          LiveRange LR(getLoadIndex(index), getUseIndex(index),
        if (HasUse) {
          LiveRange LR(getLoadIndex(index), getUseIndex(index),
-                     nI.getNextValue(~0U, 0, getVNInfoAllocator()));
-        DOUT << " +" << LR;
+                     nI.getNextValue(MachineInstrIndex(), 0, false,
+                                     getVNInfoAllocator()));
+        DEBUG(errs() << " +" << LR);
          nI.addRange(LR);
          vrm.addRestorePoint(NewVReg, MI);
        }
        if (HasDef) {
          LiveRange LR(getDefIndex(index), getStoreIndex(index),
          nI.addRange(LR);
          vrm.addRestorePoint(NewVReg, MI);
        }
        if (HasDef) {
          LiveRange LR(getDefIndex(index), getStoreIndex(index),
-                     nI.getNextValue(~0U, 0, getVNInfoAllocator()));
-        DOUT << " +" << LR;
+                     nI.getNextValue(MachineInstrIndex(), 0, false,
+                                     getVNInfoAllocator()));
+        DEBUG(errs() << " +" << LR);
          nI.addRange(LR);
          vrm.addSpillPoint(NewVReg, true, MI);
        }
        
        added.push_back(&nI);
          
          nI.addRange(LR);
          vrm.addSpillPoint(NewVReg, true, MI);
        }
        
        added.push_back(&nI);
          
-      DOUT << "\t\t\t\tadded new interval: ";
-      DEBUG(nI.dump());
-      DOUT << '\n';
-      
-      unsigned loopDepth = loopInfo->getLoopDepth(MI->getParent());
-      if (HasUse) {
-        if (HasDef)
-          SSWeight += getSpillWeight(true, true, loopDepth);
-        else
-          SSWeight += getSpillWeight(false, true, loopDepth);
-      } else
-        SSWeight += getSpillWeight(true, false, loopDepth);
+      DEBUG({
+          errs() << "\t\t\t\tadded new interval: ";
+          nI.dump();
+          errs() << '\n';
+        });
      }
      
      
      RI = mri_->reg_begin(li.reg);
    }
  
      }
      
      
      RI = mri_->reg_begin(li.reg);
    }
  
-  // Clients expect the new intervals to be returned in sorted order.
-  std::sort(added.begin(), added.end(), LISorter());
-
    return added;
  }
  
  std::vector<LiveInterval*> LiveIntervals::
  addIntervalsForSpills(const LiveInterval &li,
                        SmallVectorImpl<LiveInterval*> &SpillIs,
    return added;
  }
  
  std::vector<LiveInterval*> LiveIntervals::
  addIntervalsForSpills(const LiveInterval &li,
                        SmallVectorImpl<LiveInterval*> &SpillIs,
-                      const MachineLoopInfo *loopInfo, VirtRegMap &vrm,
-                      float &SSWeight) {
+                      const MachineLoopInfo *loopInfo, VirtRegMap &vrm) {
    
    if (EnableFastSpilling)
    
    if (EnableFastSpilling)
-    return addIntervalsForSpillsFast(li, loopInfo, vrm, SSWeight);
+    return addIntervalsForSpillsFast(li, loopInfo, vrm);
    
    assert(li.weight != HUGE_VALF &&
           "attempt to spill already spilled interval!");
  
    
    assert(li.weight != HUGE_VALF &&
           "attempt to spill already spilled interval!");
  
-  DOUT << "\t\t\t\tadding intervals for spills for interval: ";
-  li.print(DOUT, tri_);
-  DOUT << '\n';
-
-  // Spill slot weight.
-  SSWeight = 0.0f;
+  DEBUG({
+      errs() << "\t\t\t\tadding intervals for spills for interval: ";
+      li.print(errs(), tri_);
+      errs() << '\n';
+    });
  
    // Each bit specify whether a spill is required in the MBB.
    BitVector SpillMBBs(mf_->getNumBlockIDs());
  
    // Each bit specify whether a spill is required in the MBB.
    BitVector SpillMBBs(mf_->getNumBlockIDs());
@@ -1887,8 +2356,8 @@ addIntervalsForSpills(const LiveInterval &li,
    if (vrm.getPreSplitReg(li.reg)) {
      vrm.setIsSplitFromReg(li.reg, 0);
      // Unset the split kill marker on the last use.
    if (vrm.getPreSplitReg(li.reg)) {
      vrm.setIsSplitFromReg(li.reg, 0);
      // Unset the split kill marker on the last use.
-    unsigned KillIdx = vrm.getKillPoint(li.reg);
-    if (KillIdx) {
+    MachineInstrIndex KillIdx = vrm.getKillPoint(li.reg);
+    if (KillIdx != MachineInstrIndex()) {
        MachineInstr *KillMI = getInstructionFromIndex(KillIdx);
        assert(KillMI && "Last use disappeared?");
        int KillOp = KillMI->findRegisterUseOperandIdx(li.reg, true);
        MachineInstr *KillMI = getInstructionFromIndex(KillIdx);
        assert(KillMI && "Last use disappeared?");
        int KillOp = KillMI->findRegisterUseOperandIdx(li.reg, true);
@@ -1917,25 +2386,22 @@ addIntervalsForSpills(const LiveInterval &li,
                               Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                               false, vrm, rc, ReMatIds, loopInfo,
                               SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
                               Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                               false, vrm, rc, ReMatIds, loopInfo,
                               SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
-                             MBBVRegsMap, NewLIs, SSWeight);
+                             MBBVRegsMap, NewLIs);
        } else {
          rewriteInstructionsForSpills(li, false, I, NULL, 0,
                               Slot, 0, false, false, false,
                               false, vrm, rc, ReMatIds, loopInfo,
                               SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
        } else {
          rewriteInstructionsForSpills(li, false, I, NULL, 0,
                               Slot, 0, false, false, false,
                               false, vrm, rc, ReMatIds, loopInfo,
                               SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
-                             MBBVRegsMap, NewLIs, SSWeight);
+                             MBBVRegsMap, NewLIs);
        }
        IsFirstRange = false;
      }
  
        }
        IsFirstRange = false;
      }
  
-    SSWeight = 0.0f;  // Already accounted for when split.
      handleSpilledImpDefs(li, vrm, rc, NewLIs);
      return NewLIs;
    }
  
      handleSpilledImpDefs(li, vrm, rc, NewLIs);
      return NewLIs;
    }
  
-  bool TrySplit = SplitAtBB && !intervalIsInOneMBB(li);
-  if (SplitLimit != -1 && (int)numSplits >= SplitLimit)
-    TrySplit = false;
+  bool TrySplit = !intervalIsInOneMBB(li);
    if (TrySplit)
      ++numSplits;
    bool NeedStackSlot = false;
    if (TrySplit)
      ++numSplits;
    bool NeedStackSlot = false;
@@ -1943,23 +2409,22 @@ addIntervalsForSpills(const LiveInterval &li,
         i != e; ++i) {
      const VNInfo *VNI = *i;
      unsigned VN = VNI->id;
         i != e; ++i) {
      const VNInfo *VNI = *i;
      unsigned VN = VNI->id;
-    unsigned DefIdx = VNI->def;
-    if (DefIdx == ~1U)
+    if (VNI->isUnused())
        continue; // Dead val#.
      // Is the def for the val# rematerializable?
        continue; // Dead val#.
      // Is the def for the val# rematerializable?
-    MachineInstr *ReMatDefMI = (DefIdx == ~0u)
-      ? 0 : getInstructionFromIndex(DefIdx);
+    MachineInstr *ReMatDefMI = VNI->isDefAccurate()
+      ? getInstructionFromIndex(VNI->def) : 0;
      bool dummy;
      if (ReMatDefMI && isReMaterializable(li, VNI, ReMatDefMI, SpillIs, dummy)) {
        // Remember how to remat the def of this val#.
        ReMatOrigDefs[VN] = ReMatDefMI;
        // Original def may be modified so we have to make a copy here.
        MachineInstr *Clone = mf_->CloneMachineInstr(ReMatDefMI);
      bool dummy;
      if (ReMatDefMI && isReMaterializable(li, VNI, ReMatDefMI, SpillIs, dummy)) {
        // Remember how to remat the def of this val#.
        ReMatOrigDefs[VN] = ReMatDefMI;
        // Original def may be modified so we have to make a copy here.
        MachineInstr *Clone = mf_->CloneMachineInstr(ReMatDefMI);
-      ClonedMIs.push_back(Clone);
+      CloneMIs.push_back(Clone);
        ReMatDefs[VN] = Clone;
  
        bool CanDelete = true;
        ReMatDefs[VN] = Clone;
  
        bool CanDelete = true;
-      if (VNI->hasPHIKill) {
+      if (VNI->hasPHIKill()) {
          // A kill is a phi node, not all of its uses can be rematerialized.
          // It must not be deleted.
          CanDelete = false;
          // A kill is a phi node, not all of its uses can be rematerialized.
          // It must not be deleted.
          CanDelete = false;
@@ -2002,7 +2467,7 @@ addIntervalsForSpills(const LiveInterval &li,
                                 Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                                 CanDelete, vrm, rc, ReMatIds, loopInfo,
                                 SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
                                 Slot, LdSlot, isLoad, isLoadSS, DefIsReMat,
                                 CanDelete, vrm, rc, ReMatIds, loopInfo,
                                 SpillMBBs, SpillIdxes, RestoreMBBs, RestoreIdxes,
-                               MBBVRegsMap, NewLIs, SSWeight);
+                               MBBVRegsMap, NewLIs);
    }
  
    // Insert spills / restores if we are splitting.
    }
  
    // Insert spills / restores if we are splitting.
@@ -2016,11 +2481,9 @@ addIntervalsForSpills(const LiveInterval &li,
    if (NeedStackSlot) {
      int Id = SpillMBBs.find_first();
      while (Id != -1) {
    if (NeedStackSlot) {
      int Id = SpillMBBs.find_first();
      while (Id != -1) {
-      MachineBasicBlock *MBB = mf_->getBlockNumbered(Id);
-      unsigned loopDepth = loopInfo->getLoopDepth(MBB);
        std::vector<SRInfo> &spills = SpillIdxes[Id];
        for (unsigned i = 0, e = spills.size(); i != e; ++i) {
        std::vector<SRInfo> &spills = SpillIdxes[Id];
        for (unsigned i = 0, e = spills.size(); i != e; ++i) {
-        int index = spills[i].index;
+        MachineInstrIndex index = spills[i].index;
          unsigned VReg = spills[i].vreg;
          LiveInterval &nI = getOrCreateInterval(VReg);
          bool isReMat = vrm.isReMaterialized(VReg);
          unsigned VReg = spills[i].vreg;
          LiveInterval &nI = getOrCreateInterval(VReg);
          bool isReMat = vrm.isReMaterialized(VReg);
@@ -2058,7 +2521,7 @@ addIntervalsForSpills(const LiveInterval &li,
              if (FoundUse) {
                // Also folded uses, do not issue a load.
                eraseRestoreInfo(Id, index, VReg, RestoreMBBs, RestoreIdxes);
              if (FoundUse) {
                // Also folded uses, do not issue a load.
                eraseRestoreInfo(Id, index, VReg, RestoreMBBs, RestoreIdxes);
-              nI.removeRange(getLoadIndex(index), getUseIndex(index)+1);
+              nI.removeRange(getLoadIndex(index), getNextSlot(getUseIndex(index)));
              }
              nI.removeRange(getDefIndex(index), getStoreIndex(index));
            }
              }
              nI.removeRange(getDefIndex(index), getStoreIndex(index));
            }
@@ -2074,10 +2537,6 @@ addIntervalsForSpills(const LiveInterval &li,
            if (isKill)
              AddedKill.insert(&nI);
          }
            if (isKill)
              AddedKill.insert(&nI);
          }
-
-        // Update spill slot weight.
-        if (!isReMat)
-          SSWeight += getSpillWeight(true, false, loopDepth);
        }
        Id = SpillMBBs.find_next(Id);
      }
        }
        Id = SpillMBBs.find_next(Id);
      }
@@ -2085,13 +2544,10 @@ addIntervalsForSpills(const LiveInterval &li,
  
    int Id = RestoreMBBs.find_first();
    while (Id != -1) {
  
    int Id = RestoreMBBs.find_first();
    while (Id != -1) {
-    MachineBasicBlock *MBB = mf_->getBlockNumbered(Id);
-    unsigned loopDepth = loopInfo->getLoopDepth(MBB);
-
      std::vector<SRInfo> &restores = RestoreIdxes[Id];
      for (unsigned i = 0, e = restores.size(); i != e; ++i) {
      std::vector<SRInfo> &restores = RestoreIdxes[Id];
      for (unsigned i = 0, e = restores.size(); i != e; ++i) {
-      int index = restores[i].index;
-      if (index == -1)
+      MachineInstrIndex index = restores[i].index;
+      if (index == MachineInstrIndex())
          continue;
        unsigned VReg = restores[i].vreg;
        LiveInterval &nI = getOrCreateInterval(VReg);
          continue;
        unsigned VReg = restores[i].vreg;
        LiveInterval &nI = getOrCreateInterval(VReg);
@@ -2146,13 +2602,9 @@ addIntervalsForSpills(const LiveInterval &li,
        // If folding is not possible / failed, then tell the spiller to issue a
        // load / rematerialization for us.
        if (Folded)
        // If folding is not possible / failed, then tell the spiller to issue a
        // load / rematerialization for us.
        if (Folded)
-        nI.removeRange(getLoadIndex(index), getUseIndex(index)+1);
+        nI.removeRange(getLoadIndex(index), getNextSlot(getUseIndex(index)));
        else
          vrm.addRestorePoint(VReg, MI);
        else
          vrm.addRestorePoint(VReg, MI);
-
-      // Update spill slot weight.
-      if (!isReMat)
-        SSWeight += getSpillWeight(false, true, loopDepth);
      }
      Id = RestoreMBBs.find_next(Id);
    }
      }
      Id = RestoreMBBs.find_next(Id);
    }
@@ -2166,7 +2618,7 @@ addIntervalsForSpills(const LiveInterval &li,
        LI->weight /= InstrSlots::NUM * getApproximateInstructionCount(*LI);
        if (!AddedKill.count(LI)) {
          LiveRange *LR = &LI->ranges[LI->ranges.size()-1];
        LI->weight /= InstrSlots::NUM * getApproximateInstructionCount(*LI);
        if (!AddedKill.count(LI)) {
          LiveRange *LR = &LI->ranges[LI->ranges.size()-1];
-        unsigned LastUseIdx = getBaseIndex(LR->end);
+        MachineInstrIndex LastUseIdx = getBaseIndex(LR->end);
          MachineInstr *LastUse = getInstructionFromIndex(LastUseIdx);
          int UseIdx = LastUse->findRegisterUseOperandIdx(LI->reg, false);
          assert(UseIdx != -1);
          MachineInstr *LastUse = getInstructionFromIndex(LastUseIdx);
          int UseIdx = LastUse->findRegisterUseOperandIdx(LI->reg, false);
          assert(UseIdx != -1);
@@ -2217,7 +2669,7 @@ unsigned LiveIntervals::getNumConflictsWithPhysReg(const LiveInterval &li,
           E = mri_->reg_end(); I != E; ++I) {
      MachineOperand &O = I.getOperand();
      MachineInstr *MI = O.getParent();
           E = mri_->reg_end(); I != E; ++I) {
      MachineOperand &O = I.getOperand();
      MachineInstr *MI = O.getParent();
-    unsigned Index = getInstructionIndex(MI);
+    MachineInstrIndex Index = getInstructionIndex(MI);
      if (pli.liveAt(Index))
        ++NumConflicts;
    }
      if (pli.liveAt(Index))
        ++NumConflicts;
    }
@@ -2235,7 +2687,7 @@ bool LiveIntervals::spillPhysRegAroundRegDefsUses(const LiveInterval &li,
      // If there are registers which alias PhysReg, but which are not a
      // sub-register of the chosen representative super register. Assert
      // since we can't handle it yet.
      // If there are registers which alias PhysReg, but which are not a
      // sub-register of the chosen representative super register. Assert
      // since we can't handle it yet.
-    assert(*AS == SpillReg || !allocatableRegs_[*AS] ||
+    assert(*AS == SpillReg || !allocatableRegs_[*AS] || !hasInterval(*AS) ||
             tri_->isSuperRegister(*AS, SpillReg));
  
    bool Cut = false;
             tri_->isSuperRegister(*AS, SpillReg));
  
    bool Cut = false;
@@ -2248,29 +2700,31 @@ bool LiveIntervals::spillPhysRegAroundRegDefsUses(const LiveInterval &li,
      if (SeenMIs.count(MI))
        continue;
      SeenMIs.insert(MI);
      if (SeenMIs.count(MI))
        continue;
      SeenMIs.insert(MI);
-    unsigned Index = getInstructionIndex(MI);
+    MachineInstrIndex Index = getInstructionIndex(MI);
      if (pli.liveAt(Index)) {
        vrm.addEmergencySpill(SpillReg, MI);
      if (pli.liveAt(Index)) {
        vrm.addEmergencySpill(SpillReg, MI);
-      unsigned StartIdx = getLoadIndex(Index);
-      unsigned EndIdx = getStoreIndex(Index)+1;
+      MachineInstrIndex StartIdx = getLoadIndex(Index);
+      MachineInstrIndex EndIdx = getNextSlot(getStoreIndex(Index));
        if (pli.isInOneLiveRange(StartIdx, EndIdx)) {
          pli.removeRange(StartIdx, EndIdx);
          Cut = true;
        } else {
        if (pli.isInOneLiveRange(StartIdx, EndIdx)) {
          pli.removeRange(StartIdx, EndIdx);
          Cut = true;
        } else {
-        cerr << "Ran out of registers during register allocation!\n";
+        std::string msg;
+        raw_string_ostream Msg(msg);
+        Msg << "Ran out of registers during register allocation!";
          if (MI->getOpcode() == TargetInstrInfo::INLINEASM) {
          if (MI->getOpcode() == TargetInstrInfo::INLINEASM) {
-          cerr << "Please check your inline asm statement for invalid "
+          Msg << "\nPlease check your inline asm statement for invalid "
                 << "constraints:\n";
                 << "constraints:\n";
-          MI->print(cerr.stream(), tm_);
+          MI->print(Msg, tm_);
          }
          }
-        exit(1);
+        llvm_report_error(Msg.str());
        }
        for (const unsigned* AS = tri_->getSubRegisters(SpillReg); *AS; ++AS) {
          if (!hasInterval(*AS))
            continue;
          LiveInterval &spli = getInterval(*AS);
          if (spli.liveAt(Index))
        }
        for (const unsigned* AS = tri_->getSubRegisters(SpillReg); *AS; ++AS) {
          if (!hasInterval(*AS))
            continue;
          LiveInterval &spli = getInterval(*AS);
          if (spli.liveAt(Index))
-          spli.removeRange(getLoadIndex(Index), getStoreIndex(Index)+1);
+          spli.removeRange(getLoadIndex(Index), getNextSlot(getStoreIndex(Index)));
        }
      }
    }
        }
      }
    }
@@ -2278,16 +2732,18 @@ bool LiveIntervals::spillPhysRegAroundRegDefsUses(const LiveInterval &li,
  }
  
  LiveRange LiveIntervals::addLiveRangeToEndOfBlock(unsigned reg,
  }
  
  LiveRange LiveIntervals::addLiveRangeToEndOfBlock(unsigned reg,
-                                                   MachineInstr* startInst) {
+                                                  MachineInstr* startInst) {
    LiveInterval& Interval = getOrCreateInterval(reg);
    VNInfo* VN = Interval.getNextValue(
    LiveInterval& Interval = getOrCreateInterval(reg);
    VNInfo* VN = Interval.getNextValue(
-            getInstructionIndex(startInst) + InstrSlots::DEF,
-            startInst, getVNInfoAllocator());
-  VN->hasPHIKill = true;
-  VN->kills.push_back(getMBBEndIdx(startInst->getParent()));
-  LiveRange LR(getInstructionIndex(startInst) + InstrSlots::DEF,
-               getMBBEndIdx(startInst->getParent()) + 1, VN);
+    MachineInstrIndex(getInstructionIndex(startInst), MachineInstrIndex::DEF),
+    startInst, true, getVNInfoAllocator());
+  VN->setHasPHIKill(true);
+  VN->kills.push_back(terminatorGaps[startInst->getParent()]);
+  LiveRange LR(
+    MachineInstrIndex(getInstructionIndex(startInst), MachineInstrIndex::DEF),
+    getNextSlot(getMBBEndIdx(startInst->getParent())), VN);
    Interval.addRange(LR);
    
    return LR;
  }
    Interval.addRange(LR);
    
    return LR;
  }
+