Fix thinko. Cmp can be the first instruction in a MBB.
authorEvan Cheng <evan.cheng@apple.com>
Tue, 15 Feb 2011 05:00:24 +0000 (05:00 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Tue, 15 Feb 2011 05:00:24 +0000 (05:00 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@125552 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/PeepholeOptimizer.cpp

index 3474f755b8ef1e8e6407b4a9053f203b7840fe25..5d7123caa0175f482e442fe6299d6b47e263a065 100644 (file)
@@ -331,7 +331,8 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
     ImmDefRegs.clear();
     ImmDefMIs.clear();
 
-    MachineBasicBlock::iterator PMII = I->begin();
+    bool First = true;
+    MachineBasicBlock::iterator PMII;
     for (MachineBasicBlock::iterator
            MII = I->begin(), MIE = I->end(); MII != MIE; ) {
       MachineInstr *MI = &*MII;
@@ -348,7 +349,7 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
         if (OptimizeCmpInstr(MI, MBB)) {
           // MI is deleted.
           Changed = true;
-          MII = llvm::next(PMII);
+          MII = First ? I->begin() : llvm::next(PMII);
           continue;
         }
       }
@@ -360,6 +361,8 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
         if (SeenMoveImm)
           Changed |= FoldImmediate(MI, MBB, ImmDefRegs, ImmDefMIs);
       }
+
+      First = false;
       PMII = MII;
       ++MII;
     }