summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Analysis/ScalarEvolution.cpp
diff options
context:
space:
mode:
authorBrendon Cahoon <bcahoon@codeaurora.org>2015-04-22 15:06:40 +0000
committerBrendon Cahoon <bcahoon@codeaurora.org>2015-04-22 15:06:40 +0000
commitf9751ad1b09ab1eecbf62a43a8ff8fc62a0963dd (patch)
tree990353ce46fd560e8d7e73af35013754054ba4d0 /llvm/lib/Analysis/ScalarEvolution.cpp
parent60785973701f5558f167ab2f584b19bf5203ae25 (diff)
downloadbcm5719-llvm-f9751ad1b09ab1eecbf62a43a8ff8fc62a0963dd.tar.gz
bcm5719-llvm-f9751ad1b09ab1eecbf62a43a8ff8fc62a0963dd.zip
Fix a type mismatch assert in SCEV division
An assert was triggered when attempting to create a new SCEV with operands of different types in the visitAddRecExpr. In this test case, the operand types of the numerator and denominator are different. The SCEV division code should generate a conservative answer when this happens. Differential Revision: http://reviews.llvm.org/D9021 llvm-svn: 235511
Diffstat (limited to 'llvm/lib/Analysis/ScalarEvolution.cpp')
-rw-r--r--llvm/lib/Analysis/ScalarEvolution.cpp8
1 files changed, 8 insertions, 0 deletions
diff --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp
index d88b0268651..9d99b8f77c9 100644
--- a/llvm/lib/Analysis/ScalarEvolution.cpp
+++ b/llvm/lib/Analysis/ScalarEvolution.cpp
@@ -795,6 +795,14 @@ public:
assert(Numerator->isAffine() && "Numerator should be affine");
divide(SE, Numerator->getStart(), Denominator, &StartQ, &StartR);
divide(SE, Numerator->getStepRecurrence(SE), Denominator, &StepQ, &StepR);
+ // Bail out if the types do not match.
+ Type *Ty = Denominator->getType();
+ if (Ty != StartQ->getType() || Ty != StartR->getType() ||
+ Ty != StepQ->getType() || Ty != StepR->getType()) {
+ Quotient = Zero;
+ Remainder = Numerator;
+ return;
+ }
Quotient = SE.getAddRecExpr(StartQ, StepQ, Numerator->getLoop(),
Numerator->getNoWrapFlags());
Remainder = SE.getAddRecExpr(StartR, StepR, Numerator->getLoop(),
OpenPOWER on IntegriCloud