diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-08-13 23:36:35 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-08-13 23:36:35 +0000 |
commit | 2a2f14d7abc76de65cb7c8d4599e69392c3bca7d (patch) | |
tree | 2940a213fd8cd2d8568e51c4593688e6f54bc999 /llvm/lib | |
parent | e62aaf2303a11e53741b0dacb6d93f29ee706afc (diff) | |
download | bcm5719-llvm-2a2f14d7abc76de65cb7c8d4599e69392c3bca7d.tar.gz bcm5719-llvm-2a2f14d7abc76de65cb7c8d4599e69392c3bca7d.zip |
[IRCE] Be resilient in the face of non-simplified loops
Loops containing `indirectbr` may not be in simplified form, even after
running LoopSimplify. Reject then gracefully, instead of tripping an
assert.
llvm-svn: 278611
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp b/llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp index b3bd1f01232..3f9432e4651 100644 --- a/llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp +++ b/llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp @@ -622,9 +622,12 @@ static bool CanBeSMin(ScalarEvolution &SE, const SCEV *S) { Optional<LoopStructure> LoopStructure::parseLoopStructure(ScalarEvolution &SE, BranchProbabilityInfo &BPI, Loop &L, const char *&FailureReason) { - assert(L.isLoopSimplifyForm() && "should follow from addRequired<>"); + if (!L.isLoopSimplifyForm()) + return None; BasicBlock *Latch = L.getLoopLatch(); + assert(Latch && "Simplified loops only have one latch!"); + if (!L.isLoopExiting(Latch)) { FailureReason = "no loop latch"; return None; |