[llvm] [LV] Support tail-folded epilogue loops (PR #208764)
Hassnaa Hamdi via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 7 06:50:39 PDT 2026
================
@@ -5299,12 +5596,91 @@ void LoopVectorizationPlanner::plan(ElementCount UserVF, unsigned UserIC) {
CM->collectNonVectorizedAndSetWideningDecisions(VF);
}
- buildVPlans(*VPlan1, ElementCount::getFixed(1), MaxFactors.FixedVF);
- buildVPlans(*VPlan1, ElementCount::getScalable(1), MaxFactors.ScalableVF);
+ buildVPlans(*VPlan1, ElementCount::getFixed(1), MaxFactors.FixedVF, *CM);
+ buildVPlans(*VPlan1, ElementCount::getScalable(1), MaxFactors.ScalableVF,
+ *CM);
LLVM_DEBUG(printPlans(dbgs()));
}
+bool LoopVectorizationPlanner::planForEpilogueTF() {
+ if (!EpilogueTfCM)
+ return false;
+ assert(EpilogueTfCM->preferTailFoldedLoop() &&
+ "Epilogue tail-folding is expected to be enabled");
+
+ LLVM_DEBUG(dbgs() << "LV: plan for tail-folded epilogue\n");
+
+ EpilogueTfCM->ValuesToIgnore.insert_range(CM->ValuesToIgnore);
+ EpilogueTfCM->VecValuesToIgnore.insert_range(CM->VecValuesToIgnore);
+
+ FixedScalableVFPair MaxFactors =
+ EpilogueTfCM->computeMaxVF(EpilogueVectorizationForceVF, /*UserIC*/ 1);
+ if (!MaxFactors || !EpilogueTfCM->foldTailByMasking()) {
+ // Cases that should not to be vectorized or tail-folded.
+ reportVectorizationInfo("This case of epilogue loop can't be tail-folded",
+ "InvalidTailFoldedEpilogue", ORE, OrigLoop);
+ return false;
+ }
+
+ auto VPlan1 = tryToBuildVPlan1(*EpilogueTfCM);
+
+ // If we're here, the main loop's initial VPlan was built successfully.
+ // Building one for the tail-folded loop should therefore also succeed, since
+ // nothing tail-folding-specific happens yet at this point. Still check below
+ // to catch any unexpected failure.
+ if (!VPlan1) {
+ reportVectorizationInfo(
+ "Failed to build initial tail-folded epilogue VPlan",
+ "InvalidTailFoldedEpilogue", ORE, OrigLoop);
+ assert(false && "Failed to build initial tail-folded epilogue VPlan");
+ return false;
+ }
+
+ if (!useMaskedInterleavedAccesses(TTI)) {
+ LLVM_DEBUG(
+ dbgs() << "LV: Invalidate all interleaved groups due to fold-tail by "
+ "masking which requires masked-interleaved support.\n");
+ if (EpilogueTfCM->InterleaveInfo.invalidateGroups())
+ // Invalidating interleave groups also requires invalidating all decisions
+ // based on them, which includes widening decisions and uniform and scalar
+ // values.
+ EpilogueTfCM->invalidateCostModelingDecisions();
+ }
+ Legal->prepareToFoldTailByMasking();
+
+ // Collect the instructions (and their associated costs) that will be more
+ // profitable to scalarize.
+ EpilogueTfCM->collectNonVectorizedAndSetWideningDecisions(
+ EpilogueVectorizationForceVF);
+
+ size_t NumPlansBefore = VPlans.size();
+ buildVPlans(*VPlan1, EpilogueVectorizationForceVF,
+ EpilogueVectorizationForceVF, *EpilogueTfCM);
+
+ // Check that a vplan is successfully built:
+ if (VPlans.size() == NumPlansBefore ||
+ VPlans.back()->getSingleVF() != EpilogueVectorizationForceVF ||
+ !VPlans.back()->hasTailFolded()) {
+ reportVectorizationInfo(
+ "Failed to build a valid tail-folded epilogue VPlan",
+ "InvalidTailFoldedEpilogue", ORE, OrigLoop);
+ return false;
+ }
+
+ if (!cost(*VPlans.back(), EpilogueVectorizationForceVF, /*RU=*/nullptr,
----------------
hassnaaHamdi wrote:
Yeah, the comment was about not checking the result. I will add a test covering it.
https://github.com/llvm/llvm-project/pull/208764
More information about the llvm-commits
mailing list