Find anchoring end points for repairIntervalsInRange and repairIndexesInRange
authorCameron Zwarich <zwarich@apple.com>
Wed, 20 Feb 2013 22:10:00 +0000 (22:10 +0000)
committerCameron Zwarich <zwarich@apple.com>
Wed, 20 Feb 2013 22:10:00 +0000 (22:10 +0000)
automatically.

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

lib/CodeGen/LiveIntervalAnalysis.cpp
lib/CodeGen/SlotIndexes.cpp
lib/CodeGen/TwoAddressInstructionPass.cpp

index e07922b54c9be6cf5f5ef8b717d32f0f484f36da..a716e8bbab03cb41bb5ee4de4d79ec369eda1987 100644 (file)
@@ -1038,6 +1038,13 @@ LiveIntervals::repairIntervalsInRange(MachineBasicBlock *MBB,
                                       MachineBasicBlock::iterator Begin,
                                       MachineBasicBlock::iterator End,
                                       ArrayRef<unsigned> OrigRegs) {
+  // Find anchor points, which are at the beginning/end of blocks or at
+  // instructions that already have indexes.
+  while (Begin != MBB->begin() && !Indexes->hasIndex(Begin))
+    --Begin;
+  while (End != MBB->end() && !Indexes->hasIndex(End))
+    ++End;
+
   SlotIndex endIdx;
   if (End == MBB->end())
     endIdx = getMBBEndIdx(MBB).getPrevSlot();
index b4e562e45b27c8d831e4f30f5e819bf27b6b63f6..f2937941ee9fc4ae048d100e2debdf1a6d680c26 100644 (file)
@@ -146,6 +146,15 @@ void SlotIndexes::renumberIndexes(IndexList::iterator curItr) {
 void SlotIndexes::repairIndexesInRange(MachineBasicBlock *MBB,
                                        MachineBasicBlock::iterator Begin,
                                        MachineBasicBlock::iterator End) {
+  // FIXME: Is this really necessary? The only caller repairIntervalsForRange()
+  // does the same thing.
+  // Find anchor points, which are at the beginning/end of blocks or at
+  // instructions that already have indexes.
+  while (Begin != MBB->begin() && !hasIndex(Begin))
+    --Begin;
+  while (End != MBB->end() && !hasIndex(End))
+    ++End;
+
   bool includeStart = (Begin == MBB->begin());
   SlotIndex startIdx;
   if (includeStart)
index 99d36071cfa43dab1bbb60d581e6aa6b58cae3f0..43d06557084e413bc9cdd515c1a2c41a37d2e527 100644 (file)
@@ -1150,15 +1150,8 @@ tryInstructionTransform(MachineBasicBlock::iterator &mi,
             LV->addVirtualRegisterKilled(Reg, NewMIs[1]);
           }
 
-          MachineBasicBlock::iterator Begin;
-          MachineBasicBlock::iterator End;
           SmallVector<unsigned, 4> OrigRegs;
           if (LIS) {
-            Begin = MachineBasicBlock::iterator(NewMIs[0]);
-            if (Begin != MBB->begin())
-              --Begin;
-            End = llvm::next(MachineBasicBlock::iterator(MI));
-
             for (MachineInstr::const_mop_iterator MOI = MI.operands_begin(),
                  MOE = MI.operands_end(); MOI != MOE; ++MOI) {
               if (MOI->isReg())
@@ -1169,8 +1162,11 @@ tryInstructionTransform(MachineBasicBlock::iterator &mi,
           MI.eraseFromParent();
 
           // Update LiveIntervals.
-          if (LIS)
+          if (LIS) {
+            MachineBasicBlock::iterator Begin(NewMIs[0]);
+            MachineBasicBlock::iterator End(NewMIs[1]);
             LIS->repairIntervalsInRange(MBB, Begin, End, OrigRegs);
+          }
 
           mi = NewMIs[1];
           if (TransformSuccess)
@@ -1576,9 +1572,6 @@ eliminateRegSequence(MachineBasicBlock::iterator &MBBI) {
   }
 
   // Udpate LiveIntervals.
-  if (LIS) {
-    if (MBBI != MBB->begin())
-      --MBBI;
+  if (LIS)
     LIS->repairIntervalsInRange(MBB, MBBI, EndMBBI, OrigRegs);
-  }
 }