diff options
| author | Dan Gohman <gohman@apple.com> | 2009-11-20 20:51:18 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2009-11-20 20:51:18 +0000 |
| commit | a58a04921deba911d6ead8d24f495cec234681c1 (patch) | |
| tree | b051c1c24f3d53ceea308c0a6ca0ddb599f21e57 /lib/Transforms | |
| parent | 6a2392131e2e5b8c4123d69cc959831b4fe719ca (diff) | |
| download | external_llvm-a58a04921deba911d6ead8d24f495cec234681c1.zip external_llvm-a58a04921deba911d6ead8d24f495cec234681c1.tar.gz external_llvm-a58a04921deba911d6ead8d24f495cec234681c1.tar.bz2 | |
Make Loop::getLoopLatch() work on loops which don't have preheaders, as
it may be used in contexts where preheader insertion may have failed due
to an indirectbr.
Make LoopSimplify's LoopSimplify::SeparateNestedLoop properly fail in
the case that it would require splitting an indirectbr edge.
These fix PR5502.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@89484 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Utils/LoopSimplify.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/Transforms/Utils/LoopSimplify.cpp b/lib/Transforms/Utils/LoopSimplify.cpp index 44a2c1f..690972d 100644 --- a/lib/Transforms/Utils/LoopSimplify.cpp +++ b/lib/Transforms/Utils/LoopSimplify.cpp @@ -477,8 +477,13 @@ Loop *LoopSimplify::SeparateNestedLoop(Loop *L, LPPassManager &LPM) { SmallVector<BasicBlock*, 8> OuterLoopPreds; for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) if (PN->getIncomingValue(i) != PN || - !L->contains(PN->getIncomingBlock(i))) + !L->contains(PN->getIncomingBlock(i))) { + // We can't split indirectbr edges. + if (isa<IndirectBrInst>(PN->getIncomingBlock(i)->getTerminator())) + return 0; + OuterLoopPreds.push_back(PN->getIncomingBlock(i)); + } BasicBlock *Header = L->getHeader(); BasicBlock *NewBB = SplitBlockPredecessors(Header, &OuterLoopPreds[0], |