diff options
author | Cameron Zwarich <zwarich@apple.com> | 2013-02-11 09:24:45 +0000 |
---|---|---|
committer | Cameron Zwarich <zwarich@apple.com> | 2013-02-11 09:24:45 +0000 |
commit | ba378cea72f79989807c13a34dd23b9918cc340e (patch) | |
tree | ec821308ae26209c61d5c0f3efaf4fe066eb2065 /llvm/lib/CodeGen/MachineBasicBlock.cpp | |
parent | 02e742fbe6298d63280b235a6c3a2d113b52df5c (diff) | |
download | llvm-ba378cea72f79989807c13a34dd23b9918cc340e.zip llvm-ba378cea72f79989807c13a34dd23b9918cc340e.tar.gz llvm-ba378cea72f79989807c13a34dd23b9918cc340e.tar.bz2 |
Update SlotIndexes after updateTerminator() possibly removes instructions. I am
really trying to avoid piping SlotIndexes through to RemoveBranch() and friends.
llvm-svn: 174869
Diffstat (limited to 'llvm/lib/CodeGen/MachineBasicBlock.cpp')
-rw-r--r-- | llvm/lib/CodeGen/MachineBasicBlock.cpp | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/llvm/lib/CodeGen/MachineBasicBlock.cpp b/llvm/lib/CodeGen/MachineBasicBlock.cpp index 2534a74..7457cd5 100644 --- a/llvm/lib/CodeGen/MachineBasicBlock.cpp +++ b/llvm/lib/CodeGen/MachineBasicBlock.cpp @@ -693,8 +693,32 @@ MachineBasicBlock::SplitCriticalEdge(MachineBasicBlock *Succ, Pass *P) { } ReplaceUsesOfBlockWith(Succ, NMBB); + + // If updateTerminator() removes instructions, we need to remove them from + // SlotIndexes. + SmallVector<MachineInstr*, 4> Terminators; + if (Indexes) { + for (instr_iterator I = getFirstInstrTerminator(), E = instr_end(); + I != E; ++I) + Terminators.push_back(I); + } + updateTerminator(); + if (Indexes) { + SmallVector<MachineInstr*, 4> NewTerminators; + for (instr_iterator I = getFirstInstrTerminator(), E = instr_end(); + I != E; ++I) + NewTerminators.push_back(I); + + for (SmallVectorImpl<MachineInstr*>::iterator I = Terminators.begin(), + E = Terminators.end(); I != E; ++I) { + if (std::find(NewTerminators.begin(), NewTerminators.end(), *I) == + NewTerminators.end()) + Indexes->removeMachineInstrFromMaps(*I); + } + } + // Insert unconditional "jump Succ" instruction in NMBB if necessary. NMBB->addSuccessor(Succ); if (!NMBB->isLayoutSuccessor(Succ)) { |