diff options
| author | Andrew Trick <atrick@apple.com> | 2011-10-04 03:50:44 +0000 |
|---|---|---|
| committer | Andrew Trick <atrick@apple.com> | 2011-10-04 03:50:44 +0000 |
| commit | f143b79b78d1d244809fa59320f2af2edf4e1a86 (patch) | |
| tree | d98851fb9036648d18eeb33137346849071a4847 /lib/Transforms/Utils | |
| parent | 2aeb8027d649320d5ad3a30d9bbeb71c783b50d4 (diff) | |
| download | external_llvm-f143b79b78d1d244809fa59320f2af2edf4e1a86.zip external_llvm-f143b79b78d1d244809fa59320f2af2edf4e1a86.tar.gz external_llvm-f143b79b78d1d244809fa59320f2af2edf4e1a86.tar.bz2 | |
LSR should avoid redundant edge splitting.
This handles the case in which LSR rewrites an IV user that is a phi and
splits critical edges originating from a switch.
Fixes <rdar://problem/6453893> LSR is not splitting edges "nicely"
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@141059 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Utils')
| -rw-r--r-- | lib/Transforms/Utils/BreakCriticalEdges.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/Transforms/Utils/BreakCriticalEdges.cpp b/lib/Transforms/Utils/BreakCriticalEdges.cpp index 4648f49..c052910 100644 --- a/lib/Transforms/Utils/BreakCriticalEdges.cpp +++ b/lib/Transforms/Utils/BreakCriticalEdges.cpp @@ -167,7 +167,8 @@ static void CreatePHIsForSplitLoopExit(SmallVectorImpl<BasicBlock *> &Preds, /// to. /// BasicBlock *llvm::SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum, - Pass *P, bool MergeIdenticalEdges) { + Pass *P, bool MergeIdenticalEdges, + bool DontDeleteUselessPhis) { if (!isCriticalEdge(TI, SuccNum, MergeIdenticalEdges)) return 0; assert(!isa<IndirectBrInst>(TI) && @@ -224,7 +225,7 @@ BasicBlock *llvm::SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum, if (TI->getSuccessor(i) != DestBB) continue; // Remove an entry for TIBB from DestBB phi nodes. - DestBB->removePredecessor(TIBB); + DestBB->removePredecessor(TIBB, DontDeleteUselessPhis); // We found another edge to DestBB, go to NewBB instead. TI->setSuccessor(i, NewBB); |