aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorian Hahn <flo@fhahn.com>2024-06-27 12:25:55 +0100
committerFlorian Hahn <flo@fhahn.com>2024-06-27 12:25:59 +0100
commit06079233f86325cb27a8197975a2bf351652e190 (patch)
tree803092879c1015a7e8d925222cbd0593c374750c
parent2731d26948384f1ec2c30ce6692c60e9414ea2ec (diff)
downloadllvm-06079233f86325cb27a8197975a2bf351652e190.zip
llvm-06079233f86325cb27a8197975a2bf351652e190.tar.gz
llvm-06079233f86325cb27a8197975a2bf351652e190.tar.bz2
[VPlan] Return std::nullopt early if plans are empty.
Fixes a crash caused by abf5969.
-rw-r--r--llvm/lib/Transforms/Vectorize/LoopVectorize.cpp2
-rw-r--r--llvm/test/Transforms/LoopVectorize/AArch64/cost-no-valid-vplans-built.ll43
2 files changed, 45 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index 1b6de49..7514b4b 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -7296,6 +7296,8 @@ LoopVectorizationPlanner::plan(ElementCount UserVF, unsigned UserIC) {
buildVPlansWithVPRecipes(ElementCount::getScalable(1), MaxFactors.ScalableVF);
LLVM_DEBUG(printPlans(dbgs()));
+ if (VPlans.empty())
+ return std::nullopt;
if (all_of(VPlans,
[](std::unique_ptr<VPlan> &P) { return P->hasScalarVFOnly(); }))
return VectorizationFactor::Disabled();
diff --git a/llvm/test/Transforms/LoopVectorize/AArch64/cost-no-valid-vplans-built.ll b/llvm/test/Transforms/LoopVectorize/AArch64/cost-no-valid-vplans-built.ll
new file mode 100644
index 0000000..4ef2ddc
--- /dev/null
+++ b/llvm/test/Transforms/LoopVectorize/AArch64/cost-no-valid-vplans-built.ll
@@ -0,0 +1,43 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 5
+; RUN: opt -p loop-vectorize -S %s | FileCheck %s
+
+target datalayout = "e-m:o-i64:64-i128:128-n32:64-S128-Fn32"
+target triple = "arm64-apple-macosx15.0.0"
+
+define void @test(i32 %N, ptr %dst) {
+; CHECK-LABEL: define void @test(
+; CHECK-SAME: i32 [[N:%.*]], ptr [[DST:%.*]]) {
+; CHECK-NEXT: [[ENTRY:.*]]:
+; CHECK-NEXT: br label %[[LOOP:.*]]
+; CHECK: [[LOOP]]:
+; CHECK-NEXT: [[IV:%.*]] = phi i32 [ 0, %[[ENTRY]] ], [ [[IV_NEXT:%.*]], %[[LOOP]] ]
+; CHECK-NEXT: [[FNEG137:%.*]] = phi float [ 0.000000e+00, %[[ENTRY]] ], [ [[FNEG1:%.*]], %[[LOOP]] ]
+; CHECK-NEXT: [[FNEG46:%.*]] = phi float [ 0.000000e+00, %[[ENTRY]] ], [ [[FNEG:%.*]], %[[LOOP]] ]
+; CHECK-NEXT: [[FNEG]] = fneg float [[FNEG137]]
+; CHECK-NEXT: [[FNEG1]] = fneg float [[FNEG46]]
+; CHECK-NEXT: [[GEP:%.*]] = getelementptr float, ptr [[DST]], i32 [[IV]]
+; CHECK-NEXT: store float [[FNEG1]], ptr [[GEP]], align 4
+; CHECK-NEXT: [[IV_NEXT]] = add i32 [[IV]], 1
+; CHECK-NEXT: [[EC:%.*]] = icmp eq i32 [[IV]], [[N]]
+; CHECK-NEXT: br i1 [[EC]], label %[[EXIT:.*]], label %[[LOOP]]
+; CHECK: [[EXIT]]:
+; CHECK-NEXT: ret void
+;
+entry:
+ br label %loop
+
+loop:
+ %iv = phi i32 [ 0, %entry ], [ %iv.next, %loop ]
+ %fneg137 = phi float [ 0.000000e+00, %entry ], [ %fneg1, %loop ]
+ %fneg46 = phi float [ 0.000000e+00, %entry ], [ %fneg, %loop ]
+ %fneg = fneg float %fneg137
+ %fneg1 = fneg float %fneg46
+ %gep = getelementptr float, ptr %dst, i32 %iv
+ store float %fneg1, ptr %gep
+ %iv.next = add i32 %iv, 1
+ %ec = icmp eq i32 %iv, %N
+ br i1 %ec, label %exit, label %loop
+
+exit:
+ ret void
+}