summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIgor Kirillov <igor.kirillov@arm.com>2023-08-10 15:35:39 +0000
committerTobias Hieta <tobias@hieta.se>2023-09-11 08:53:37 +0200
commit0250eb40f9c7f6ffddff4d32d7724f41fd5d4bce (patch)
tree9f67b16d65aa927f3a29d82b757c0d7f065e6e0a
parent[RelNotes] Add more details on the removal of the legacy optimization pipeline (diff)
downloadllvm-project-0250eb40f9c7f6ffddff4d32d7724f41fd5d4bce.tar.gz
llvm-project-0250eb40f9c7f6ffddff4d32d7724f41fd5d4bce.tar.bz2
llvm-project-0250eb40f9c7f6ffddff4d32d7724f41fd5d4bce.zip
[LoopVectorize] Pre-commit tests for D157631
Differential Revision: https://reviews.llvm.org/D157630 (cherry picked from commit 2df9ed11c51ebf6d19f7cb22c3794d1bd460ddf8)
-rw-r--r--llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll121
1 files changed, 121 insertions, 0 deletions
diff --git a/llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll b/llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll
index b953e215fa0a..5734eb6dda59 100644
--- a/llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll
+++ b/llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll
@@ -559,3 +559,124 @@ exit: ; preds = %for.body
%add.lcssa = phi i32 [ %add, %for.body ]
ret i32 %add.lcssa
}
+
+; Make sure that if there are several reductions in the loop, the order of invariant stores sank outside of the loop is preserved
+; FIXME: This tests currently shows incorrect behavior and it will fixed in the following patch
+; See https://github.com/llvm/llvm-project/issues/64047
+define void @reduc_add_mul_store_same_ptr(ptr %dst, ptr readonly %src) {
+; CHECK-LABEL: define void @reduc_add_mul_store_same_ptr
+; CHECK: middle.block:
+; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.vector.reduce.mul.v4i32(<4 x i32> [[TMP1:%.*]])
+; CHECK-NEXT: store i32 [[TMP2]], ptr %dst, align 4
+; CHECK-NEXT: [[TMP4:%.*]] = call i32 @llvm.vector.reduce.add.v4i32(<4 x i32> [[TMP3:%.*]])
+; CHECK-NEXT: store i32 [[TMP4]], ptr %dst, align 4
+;
+entry:
+ br label %for.body
+
+for.body:
+ %sum = phi i32 [ 0, %entry ], [ %sum.next, %for.body ]
+ %mul = phi i32 [ 1, %entry ], [ %mul.next, %for.body ]
+ %iv = phi i64 [ 0, %entry ], [ %iv.next, %for.body ]
+ %gep.src = getelementptr inbounds i32, ptr %src, i64 %iv
+ %0 = load i32, ptr %gep.src, align 4
+ %sum.next = add nsw i32 %sum, %0
+ store i32 %sum.next, ptr %dst, align 4
+ %mul.next = mul nsw i32 %mul, %0
+ store i32 %mul.next, ptr %dst, align 4
+ %iv.next = add nuw nsw i64 %iv, 1
+ %exitcond = icmp eq i64 %iv.next, 1000
+ br i1 %exitcond, label %exit, label %for.body
+
+exit:
+ ret void
+}
+
+define void @reduc_mul_add_store_same_ptr(ptr %dst, ptr readonly %src) {
+; CHECK-LABEL: define void @reduc_mul_add_store_same_ptr
+; CHECK: middle.block:
+; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.vector.reduce.mul.v4i32(<4 x i32> [[TMP1:%.*]])
+; CHECK-NEXT: store i32 [[TMP2]], ptr %dst, align 4
+; CHECK-NEXT: [[TMP4:%.*]] = call i32 @llvm.vector.reduce.add.v4i32(<4 x i32> [[TMP3:%.*]])
+; CHECK-NEXT: store i32 [[TMP4]], ptr %dst, align 4
+;
+entry:
+ br label %for.body
+
+for.body:
+ %sum = phi i32 [ 0, %entry ], [ %sum.next, %for.body ]
+ %mul = phi i32 [ 1, %entry ], [ %mul.next, %for.body ]
+ %iv = phi i64 [ 0, %entry ], [ %iv.next, %for.body ]
+ %gep.src = getelementptr inbounds i32, ptr %src, i64 %iv
+ %0 = load i32, ptr %gep.src, align 4
+ %mul.next = mul nsw i32 %mul, %0
+ store i32 %mul.next, ptr %dst, align 4
+ %sum.next = add nsw i32 %sum, %0
+ store i32 %sum.next, ptr %dst, align 4
+ %iv.next = add nuw nsw i64 %iv, 1
+ %exitcond = icmp eq i64 %iv.next, 1000
+ br i1 %exitcond, label %exit, label %for.body
+
+exit:
+ ret void
+}
+
+; Same as above but storing is done to two different pointers and they can be aliased
+; FIXME: This tests currently shows incorrect behavior and it will fixed in the following patch
+define void @reduc_add_mul_store_different_ptr(ptr %dst1, ptr %dst2, ptr readonly %src) {
+; CHECK-LABEL: define void @reduc_add_mul_store_different_ptr
+; CHECK: middle.block:
+; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.vector.reduce.mul.v4i32(<4 x i32> [[TMP1:%.*]])
+; CHECK-NEXT: store i32 [[TMP2]], ptr %dst2, align 4
+; CHECK-NEXT: [[TMP4:%.*]] = call i32 @llvm.vector.reduce.add.v4i32(<4 x i32> [[TMP3:%.*]])
+; CHECK-NEXT: store i32 [[TMP4]], ptr %dst1, align 4
+;
+entry:
+ br label %for.body
+
+for.body:
+ %sum = phi i32 [ 0, %entry ], [ %sum.next, %for.body ]
+ %mul = phi i32 [ 1, %entry ], [ %mul.next, %for.body ]
+ %iv = phi i64 [ 0, %entry ], [ %iv.next, %for.body ]
+ %gep.src = getelementptr inbounds i32, ptr %src, i64 %iv
+ %0 = load i32, ptr %gep.src, align 4
+ %sum.next = add nsw i32 %sum, %0
+ store i32 %sum.next, ptr %dst1, align 4
+ %mul.next = mul nsw i32 %mul, %0
+ store i32 %mul.next, ptr %dst2, align 4
+ %iv.next = add nuw nsw i64 %iv, 1
+ %exitcond = icmp eq i64 %iv.next, 1000
+ br i1 %exitcond, label %exit, label %for.body
+
+exit:
+ ret void
+}
+
+define void @reduc_mul_add_store_different_ptr(ptr %dst1, ptr %dst2, ptr readonly %src) {
+; CHECK-LABEL: define void @reduc_mul_add_store_different_ptr
+; CHECK: middle.block:
+; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.vector.reduce.mul.v4i32(<4 x i32> [[TMP1:%.*]])
+; CHECK-NEXT: store i32 [[TMP2]], ptr %dst1, align 4
+; CHECK-NEXT: [[TMP4:%.*]] = call i32 @llvm.vector.reduce.add.v4i32(<4 x i32> [[TMP3:%.*]])
+; CHECK-NEXT: store i32 [[TMP4]], ptr %dst2, align 4
+;
+entry:
+ br label %for.body
+
+for.body:
+ %sum = phi i32 [ 0, %entry ], [ %sum.next, %for.body ]
+ %mul = phi i32 [ 1, %entry ], [ %mul.next, %for.body ]
+ %iv = phi i64 [ 0, %entry ], [ %iv.next, %for.body ]
+ %gep.src = getelementptr inbounds i32, ptr %src, i64 %iv
+ %0 = load i32, ptr %gep.src, align 4
+ %mul.next = mul nsw i32 %mul, %0
+ store i32 %mul.next, ptr %dst1, align 4
+ %sum.next = add nsw i32 %sum, %0
+ store i32 %sum.next, ptr %dst2, align 4
+ %iv.next = add nuw nsw i64 %iv, 1
+ %exitcond = icmp eq i64 %iv.next, 1000
+ br i1 %exitcond, label %exit, label %for.body
+
+exit:
+ ret void
+}