[llvm] [VPlan] Connect (MemRuntime|SCEV)Check blocks as VPlan transform (NFC). (PR #143879)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 5 12:25:00 PDT 2025
================
@@ -560,6 +564,9 @@ class LoopVectorizationPlanner {
VPRecipeBuilder &RecipeBuilder,
ElementCount MinVF);
+ /// Add the runtime checks from \p RTChecks to \p VPlan.
+ void addRuntimeChecks(VPlan &Plan, GeneratedRTChecks &RTChecks) const;
----------------
ayalz wrote:
```suggestion
/// Attach the runtime checks of \p RTChecks to \p VPlan.
void attachRuntimeChecks(VPlan &Plan, GeneratedRTChecks &RTChecks) const;
```
?
The runtime checks are conceptually "there", but held detached from VPlan(s) until now for compactness, and detached from IR until VPlan execution.
This method wraps `connectRuntimeChecks()` with asserts and remarks. Suffice to have a single `VPlanTransform::attachRuntimeCheck()` with a unified assert and remark - of a conflict between a runtime check and OptForSize?
https://github.com/llvm/llvm-project/pull/143879
More information about the llvm-commits
mailing list