[llvm] 5f2b787 - NFC. Add tests exposing missing analysis invalidation in LoopPredication.
Daniil Suchkov via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 16 15:50:40 PDT 2021
Author: Daniil Suchkov
Date: 2021-09-16T22:49:55Z
New Revision: 5f2b7879f16ad5023f0684febeb0a20f7d53e4a8
URL: https://github.com/llvm/llvm-project/commit/5f2b7879f16ad5023f0684febeb0a20f7d53e4a8
DIFF: https://github.com/llvm/llvm-project/commit/5f2b7879f16ad5023f0684febeb0a20f7d53e4a8.diff
LOG: NFC. Add tests exposing missing analysis invalidation in LoopPredication.
Added:
llvm/test/Transforms/LoopPredication/invalidate-analyses.ll
Modified:
Removed:
################################################################################
diff --git a/llvm/test/Transforms/LoopPredication/invalidate-analyses.ll b/llvm/test/Transforms/LoopPredication/invalidate-analyses.ll
new file mode 100644
index 0000000000000..9c1812a9ae9d6
--- /dev/null
+++ b/llvm/test/Transforms/LoopPredication/invalidate-analyses.ll
@@ -0,0 +1,166 @@
+; RUN: opt -S -passes='require<scalar-evolution>,loop-mssa(loop-predication)' -debug-pass-manager < %s 2>&1 | FileCheck %s
+
+; FIXME: We should invalidate PreservedCFGCheckerAnalysis after LoopPredicationPass!
+; NOTE: PreservedCFGCheckerAnalysis is an arbitrary analysis that just happens
+; to be calculated before this pass and isn't preserved by it. If after
+; your change this analysis is preserved by the pass, please update this
+; test some other analysis that isn't preserved.
+
+; CHECK: Running analysis: PreservedCFGCheckerAnalysis on drop_a_wc_and_leave_early
+; CHECK: Running pass: LoopPredicationPass on Loop at depth 1 containing: %loop<header><exiting>,%guarded<exiting>,%guarded2<latch><exiting>
+; CHECK-NEXT: Running analysis: PreservedCFGCheckerAnalysis on drop_a_wc_and_leave
+; CHECK: Running pass: LoopPredicationPass on Loop at depth 1 containing: %loop<header><exiting>,%guarded<exiting>,%guarded2<latch><exiting>
+; CHECK-NEXT: Running pass: VerifierPass
+
+
+; This test makes the pass drop its attempts to optimize the exit condition in
+; `%loop` BB by using unanalyzable `%cond_0` as an exit condition.
+define i64 @drop_a_wc_and_leave_early(i64 %length, i64 %n, i1 %cond_0, i1 %cond_1) {
+; Make sure the pass has only replaced `%wc2` with `true` in the definition of `%wb_cond`.
+; CHECK-LABEL: define i64 @drop_a_wc_and_leave_early(i64 %length, i64 %n, i1 %cond_0, i1 %cond_1) {
+; CHECK-NEXT: entry:
+; CHECK-NEXT: %wc1 = call i1 @llvm.experimental.widenable.condition()
+; CHECK-NEXT: %wc2 = call i1 @llvm.experimental.widenable.condition()
+; CHECK-NEXT: %exiplicit_guard_cond = and i1 %cond_0, %wc1
+; CHECK-NEXT: br i1 %exiplicit_guard_cond, label %loop.preheader, label %deopt, !prof !0
+; CHECK: deopt:
+; CHECK-NEXT: %deoptret = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret
+; CHECK: loop.preheader:
+; CHECK-NEXT: br label %loop
+; CHECK: loop:
+; CHECK-NEXT: %i = phi i64 [ %i.next, %guarded2 ], [ 0, %loop.preheader ]
+; CHECK-NEXT: br i1 %cond_0, label %guarded, label %deopt2, !prof !0
+; CHECK: deopt2:
+; CHECK-NEXT: %deoptret2 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret2
+; CHECK: guarded:
+; CHECK-NEXT: %wb_cond = and i1 %cond_1, true
+; CHECK-NEXT: br i1 %wb_cond, label %guarded2, label %deopt3, !prof !0
+; CHECK: deopt3:
+; CHECK-NEXT: %deoptret3 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret3
+; CHECK: guarded2:
+; CHECK-NEXT: %i.next = add nuw i64 %i, 1
+; CHECK-NEXT: %continue = icmp ult i64 %i.next, %n
+; CHECK-NEXT: br i1 %continue, label %loop, label %exit
+; CHECK: exit:
+; CHECK-NEXT: ret i64 0
+; CHECK-NEXT: }
+
+entry:
+ %wc1 = call i1 @llvm.experimental.widenable.condition()
+ %wc2 = call i1 @llvm.experimental.widenable.condition()
+ %exiplicit_guard_cond = and i1 %cond_0, %wc1
+ br i1 %exiplicit_guard_cond, label %loop.preheader, label %deopt, !prof !0
+
+deopt:
+ %deoptret = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret
+
+loop.preheader:
+ br label %loop
+
+loop:
+ %i = phi i64 [ %i.next, %guarded2 ], [ 0, %loop.preheader ]
+ br i1 %cond_0, label %guarded, label %deopt2, !prof !0
+
+deopt2:
+ %deoptret2 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret2
+
+guarded:
+ %wb_cond = and i1 %cond_1, %wc2
+ br i1 %wb_cond, label %guarded2, label %deopt3, !prof !0
+
+deopt3:
+ %deoptret3 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret3
+
+guarded2:
+ %i.next = add nuw i64 %i, 1
+ %continue = icmp ult i64 %i.next, %n
+ br i1 %continue, label %loop, label %exit
+
+exit:
+ ret i64 0
+}
+
+; This test makes the pass drop its attempts to optimize the exit condition in
+; `%loop` BB by using trivial `false` as an exit condition.
+define i64 @drop_a_wc_and_leave(i64 %n, i1 %cond_0, i1 %cond_1) {
+; Make sure the pass has only replaced `%wc2` with `true` in the definition of `%wb_cond`.
+; CHECK-LABEL: define i64 @drop_a_wc_and_leave(i64 %n, i1 %cond_0, i1 %cond_1) {
+; CHECK-NEXT: entry:
+; CHECK-NEXT: %wc1 = call i1 @llvm.experimental.widenable.condition()
+; CHECK-NEXT: %wc2 = call i1 @llvm.experimental.widenable.condition()
+; CHECK-NEXT: %exiplicit_guard_cond = and i1 %cond_0, %wc1
+; CHECK-NEXT: br i1 %exiplicit_guard_cond, label %loop.preheader, label %deopt, !prof !0
+; CHECK: deopt:
+; CHECK-NEXT: %deoptret = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret
+; CHECK: loop.preheader:
+; CHECK-NEXT: br label %loop
+; CHECK: loop:
+; CHECK-NEXT: %i = phi i64 [ %i.next, %guarded2 ], [ 0, %loop.preheader ]
+; CHECK-NEXT: br i1 false, label %guarded, label %deopt2, !prof !0
+; CHECK: deopt2:
+; CHECK-NEXT: %deoptret2 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret2
+; CHECK: guarded:
+; CHECK-NEXT: %wb_cond = and i1 %cond_1, true
+; CHECK-NEXT: br i1 %wb_cond, label %guarded2, label %deopt3, !prof !0
+; CHECK: deopt3:
+; CHECK-NEXT: %deoptret3 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+; CHECK-NEXT: ret i64 %deoptret3
+; CHECK: guarded2:
+; CHECK-NEXT: %i.next = add nuw i64 %i, 1
+; CHECK-NEXT: %continue = icmp ult i64 %i.next, %n
+; CHECK-NEXT: br i1 %continue, label %loop, label %exit
+; CHECK: exit:
+; CHECK-NEXT: ret i64 0
+; CHECK-NEXT: }
+
+entry:
+ %wc1 = call i1 @llvm.experimental.widenable.condition()
+ %wc2 = call i1 @llvm.experimental.widenable.condition()
+ %exiplicit_guard_cond = and i1 %cond_0, %wc1
+ br i1 %exiplicit_guard_cond, label %loop.preheader, label %deopt, !prof !0
+
+deopt:
+ %deoptret = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret
+
+loop.preheader:
+ br label %loop
+
+loop:
+ %i = phi i64 [ %i.next, %guarded2 ], [ 0, %loop.preheader ]
+ br i1 false, label %guarded, label %deopt2, !prof !0
+
+deopt2:
+ %deoptret2 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret2
+
+guarded:
+ %wb_cond = and i1 %cond_1, %wc2
+ br i1 %wb_cond, label %guarded2, label %deopt3, !prof !0
+
+deopt3:
+ %deoptret3 = call i64 (...) @llvm.experimental.deoptimize.i64() [ "deopt"() ]
+ ret i64 %deoptret3
+
+guarded2:
+ %i.next = add nuw i64 %i, 1
+ %continue = icmp ult i64 %i.next, %n
+ br i1 %continue, label %loop, label %exit
+
+exit:
+ ret i64 0
+}
+
+
+declare i1 @llvm.experimental.widenable.condition()
+declare i64 @llvm.experimental.deoptimize.i64(...)
+
+!0 = !{!"branch_weights", i64 1048576, i64 1}
More information about the llvm-commits
mailing list