summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSanjoy Das <sanjoy@playingwithpointers.com>2016-05-03 17:50:11 +0000
committerSanjoy Das <sanjoy@playingwithpointers.com>2016-05-03 17:50:11 +0000
commit4ae3920c5b143016ccb1d10d1747f26829c449d5 (patch)
tree7cfffdc9648e53a2ceb2f5476647dca8fa94cb46
parent7e7a5a050ab4f424a2a5832567f3ee2093aa0926 (diff)
downloadbcm5719-llvm-4ae3920c5b143016ccb1d10d1747f26829c449d5.tar.gz
bcm5719-llvm-4ae3920c5b143016ccb1d10d1747f26829c449d5.zip
[LICM] Kill SCEV loop dispositions if needed
SCEV caches whether SCEV expressions are loop invariant, variant or computable. LICM breaks this cache, almost by definition; so clear the SCEV disposition cache if LICM changed anything. llvm-svn: 268408
-rw-r--r--llvm/lib/Transforms/Scalar/LICM.cpp4
-rw-r--r--llvm/test/Transforms/LICM/update-scev.ll31
2 files changed, 35 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/LICM.cpp b/llvm/lib/Transforms/Scalar/LICM.cpp
index 72f04165e66..e6ad42bb4c3 100644
--- a/llvm/lib/Transforms/Scalar/LICM.cpp
+++ b/llvm/lib/Transforms/Scalar/LICM.cpp
@@ -258,6 +258,10 @@ bool LICM::runOnLoop(Loop *L, LPPassManager &LPM) {
LoopToAliasSetMap[L] = CurAST;
else
delete CurAST;
+
+ if (Changed)
+ if (auto *SEWP = getAnalysisIfAvailable<ScalarEvolutionWrapperPass>())
+ SEWP->getSE().forgetLoopDispositions(L);
return Changed;
}
diff --git a/llvm/test/Transforms/LICM/update-scev.ll b/llvm/test/Transforms/LICM/update-scev.ll
new file mode 100644
index 00000000000..221c124c8bf
--- /dev/null
+++ b/llvm/test/Transforms/LICM/update-scev.ll
@@ -0,0 +1,31 @@
+; RUN: opt -S -licm < %s | FileCheck %s --check-prefix=IR-AFTER-TRANSFORM
+; RUN: opt -analyze -scalar-evolution -licm -scalar-evolution < %s | FileCheck %s --check-prefix=SCEV-EXPRS
+
+declare void @clobber()
+
+define void @f_0(i1* %loc) {
+; IR-AFTER-TRANSFORM-LABEL: @f_0(
+; IR-AFTER-TRANSFORM: loop.outer:
+; IR-AFTER-TRANSFORM-NEXT: call void @clobber()
+; IR-AFTER-TRANSFORM-NEXT: %cond = load i1, i1* %loc
+; IR-AFTER-TRANSFORM-NEXT: br label %loop.inner
+
+; SCEV-EXPRS: Classifying expressions for: @f_0
+; SCEV-EXPRS: Classifying expressions for: @f_0
+; SCEV-EXPRS: %cond = load i1, i1* %loc
+; SCEV-EXPRS-NEXT: --> {{.*}} LoopDispositions: { %loop.outer: Variant, %loop.inner: Invariant }
+
+entry:
+ br label %loop.outer
+
+loop.outer:
+ call void @clobber()
+ br label %loop.inner
+
+loop.inner:
+ %cond = load i1, i1* %loc
+ br i1 %cond, label %loop.inner, label %leave.inner
+
+leave.inner:
+ br label %loop.outer
+}
OpenPOWER on IntegriCloud