diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2013-06-04 17:51:58 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2013-06-04 17:51:58 +0000 |
| commit | 5a57dbef33ec2d336de2a2e9da4e477a4969fe57 (patch) | |
| tree | 2acc702f179de66ff5f061b01148bff1a8ad6087 /lib/Transforms | |
| parent | 35e7751af455f2cc559cdc2d3424b53706e09bb3 (diff) | |
| download | llvm-5a57dbef33ec2d336de2a2e9da4e477a4969fe57.tar.gz llvm-5a57dbef33ec2d336de2a2e9da4e477a4969fe57.tar.bz2 llvm-5a57dbef33ec2d336de2a2e9da4e477a4969fe57.tar.xz | |
IndVarSimplify: check if loop invariant expansion can trap
IndVarSimplify is willing to move divide instructions outside of their
loop bodies if they are invariant of the loop. However, it may not be
safe to expand them if we do not know if they can trap.
Instead, check to see if it is not safe to expand the instruction and
skip the expansion.
This fixes PR16041.
Testcase by Rafael Ávila de Espíndola.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@183239 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/IndVarSimplify.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/IndVarSimplify.cpp b/lib/Transforms/Scalar/IndVarSimplify.cpp index 8e76c78f5a..df11e92c9e 100644 --- a/lib/Transforms/Scalar/IndVarSimplify.cpp +++ b/lib/Transforms/Scalar/IndVarSimplify.cpp @@ -532,7 +532,7 @@ void IndVarSimplify::RewriteLoopExitValues(Loop *L, SCEVExpander &Rewriter) { // and varies predictably *inside* the loop. Evaluate the value it // contains when the loop exits, if possible. const SCEV *ExitValue = SE->getSCEVAtScope(Inst, L->getParentLoop()); - if (!SE->isLoopInvariant(ExitValue, L)) + if (!SE->isLoopInvariant(ExitValue, L) || !isSafeToExpand(ExitValue)) continue; // Computing the value outside of the loop brings no benefit if : |