diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2011-07-19 22:59:41 +0000 | 
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2011-07-19 22:59:41 +0000 | 
| commit | 36106040862e876a53a56f5765d6d3cd1181dd04 (patch) | |
| tree | acd4e7bd7386cc8f9bd2ca1ce648a87292004a6b /lib | |
| parent | 14c92467e79f5f151b4b53ab2b1a7fd0f7881041 (diff) | |
| download | external_llvm-36106040862e876a53a56f5765d6d3cd1181dd04.zip external_llvm-36106040862e876a53a56f5765d6d3cd1181dd04.tar.gz external_llvm-36106040862e876a53a56f5765d6d3cd1181dd04.tar.bz2  | |
PR10386: Don't try to split an edge from an indirectbr.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@135534 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/Transforms/Utils/BreakCriticalEdges.cpp | 11 | 
1 files changed, 9 insertions, 2 deletions
diff --git a/lib/Transforms/Utils/BreakCriticalEdges.cpp b/lib/Transforms/Utils/BreakCriticalEdges.cpp index 92ce500..803e431 100644 --- a/lib/Transforms/Utils/BreakCriticalEdges.cpp +++ b/lib/Transforms/Utils/BreakCriticalEdges.cpp @@ -337,6 +337,8 @@ BasicBlock *llvm::SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum,          }          // For each unique exit block... +        // FIXME: This code is functionally equivalent to the corresponding +        // loop in LoopSimplify.          SmallVector<BasicBlock *, 4> ExitBlocks;          TIL->getExitBlocks(ExitBlocks);          for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) { @@ -348,10 +350,15 @@ BasicBlock *llvm::SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum,            for (pred_iterator I = pred_begin(Exit), E = pred_end(Exit);                 I != E; ++I) {              BasicBlock *P = *I; -            if (TIL->contains(P)) +            if (TIL->contains(P)) { +              if (isa<IndirectBrInst>(P->getTerminator())) { +                Preds.clear(); +                break; +              }                Preds.push_back(P); -            else +            } else {                HasPredOutsideOfLoop = true; +            }            }            // If there are any preds not in the loop, we'll need to split            // the edges. The Preds.empty() check is needed because a block  | 
