6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-09-12 01:25:10 +00:00
Code Issues Projects Releases Wiki Activity

Use standard pattern for iterate+erase.

Browse Source 
Increment the MBB iterator at the top of the loop to properly handle the
current (and previous) instructions getting erased.

This fixes PR13625.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@162099 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Jakob Stoklund Olesen
2012-08-17 14:38:59 +00:00
parent 823573a381
commit cabc0699ea
1 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
lib/CodeGen/PeepholeOptimizer.cpp
View File

@@ -496,11 +496,11 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
ImmDefMIs.clear(); ImmDefMIs.clear();
FoldAsLoadDefReg = 0; FoldAsLoadDefReg = 0;
bool First = true;
MachineBasicBlock::iterator PMII;
for (MachineBasicBlock::iterator for (MachineBasicBlock::iterator
MII = I->begin(), MIE = I->end(); MII != MIE; ) { MII = I->begin(), MIE = I->end(); MII != MIE; ) {
MachineInstr *MI = &*MII; MachineInstr *MI = &*MII;
// We may be erasing MI below, increment MII now.
++MII;
LocalMIs.insert(MI); LocalMIs.insert(MI);
// If there exists an instruction which belongs to the following // If there exists an instruction which belongs to the following
@@ -509,7 +509,6 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
MI->isKill() || MI->isInlineAsm() || MI->isDebugValue() || MI->isKill() || MI->isInlineAsm() || MI->isDebugValue() ||
MI->hasUnmodeledSideEffects()) { MI->hasUnmodeledSideEffects()) {
FoldAsLoadDefReg = 0; FoldAsLoadDefReg = 0;
++MII;
continue; continue;
} }
if (MI->mayStore() || MI->isCall()) if (MI->mayStore() || MI->isCall())
@@ -521,7 +520,6 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
// MI is deleted. // MI is deleted.
LocalMIs.erase(MI); LocalMIs.erase(MI);
Changed = true; Changed = true;
MII = First ? I->begin() : llvm::next(PMII);
continue; continue;
} }
@@ -553,14 +551,9 @@ bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
// MI is replaced with FoldMI. // MI is replaced with FoldMI.
Changed = true; Changed = true;
PMII = FoldMI;
MII = llvm::next(PMII);
continue; continue;
} }
} }
First = false;
PMII = MII;
++MII;
} }
} }