summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Zolotukhin <mzolotukhin@apple.com>2016-05-24 00:51:01 +0000
committerMichael Zolotukhin <mzolotukhin@apple.com>2016-05-24 00:51:01 +0000
commit382294c419c41e7c28cb6987f03d5ef37590854c (patch)
treec4276692dbeebdbaf1fb9456093d59d33cc16989
parent2defe22bf32b5dbed8be4b17a1acf0b960975df7 (diff)
[LoopUnrollAnalyzer] Fix a crash in UnrolledInstAnalyzer::visitCastInst.
This fixes PR27847. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@270517 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/Analysis/LoopUnrollAnalyzer.cpp7
-rw-r--r--test/Transforms/LoopUnroll/full-unroll-crashers.ll21
2 files changed, 26 insertions, 2 deletions
diff --git a/lib/Analysis/LoopUnrollAnalyzer.cpp b/lib/Analysis/LoopUnrollAnalyzer.cpp
index a044e6bd330..413cf343c97 100644
--- a/lib/Analysis/LoopUnrollAnalyzer.cpp
+++ b/lib/Analysis/LoopUnrollAnalyzer.cpp
@@ -141,12 +141,17 @@ bool UnrolledInstAnalyzer::visitCastInst(CastInst &I) {
Constant *COp = dyn_cast<Constant>(I.getOperand(0));
if (!COp)
COp = SimplifiedValues.lookup(I.getOperand(0));
- if (COp)
+ if (COp) {
+ if (COp->getType() == I.getType()) {
+ SimplifiedValues[&I] = cast<Constant>(COp);
+ return true;
+ }
if (Constant *C =
ConstantExpr::getCast(I.getOpcode(), COp, I.getType())) {
SimplifiedValues[&I] = C;
return true;
}
+ }
return Base::visitCastInst(I);
}
diff --git a/test/Transforms/LoopUnroll/full-unroll-crashers.ll b/test/Transforms/LoopUnroll/full-unroll-crashers.ll
index e932851042a..991cb309bf4 100644
--- a/test/Transforms/LoopUnroll/full-unroll-crashers.ll
+++ b/test/Transforms/LoopUnroll/full-unroll-crashers.ll
@@ -1,5 +1,5 @@
; Check that we don't crash on corner cases.
-; RUN: opt < %s -S -loop-unroll -unroll-max-iteration-count-to-analyze=1000 -unroll-threshold=10 -unroll-percent-dynamic-cost-saved-threshold=20 -o /dev/null
+; RUN: opt < %s -S -loop-unroll -unroll-max-iteration-count-to-analyze=1000 -unroll-threshold=1 -unroll-percent-dynamic-cost-saved-threshold=20 -o /dev/null
target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
@known_constant = internal unnamed_addr constant [10 x i32] [i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1, i32 1], align 16
@@ -100,3 +100,22 @@ for.body:
for.exit:
ret <4 x i32> %r
}
+
+define void @ptrtoint_cast() optsize {
+entry:
+ br label %for.body
+
+for.body:
+ br i1 true, label %for.inc, label %if.then
+
+if.then:
+ %arraydecay = getelementptr inbounds [1 x i32], [1 x i32]* null, i64 0, i64 0
+ %x = ptrtoint i32* %arraydecay to i64
+ br label %for.inc
+
+for.inc:
+ br i1 false, label %for.body, label %for.cond.cleanup
+
+for.cond.cleanup:
+ ret void
+}