[llvm] [VPlan] Add VPValue for VF, use it for VPWidenIntOrFpInductionRecipe. (PR #95305)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 8 16:31:38 PDT 2024
================
@@ -928,8 +928,19 @@ void VPlan::prepareToExecute(Value *TripCountV, Value *VectorTripCountV,
IRBuilder<> Builder(State.CFG.PrevBB->getTerminator());
// FIXME: Model VF * UF computation completely in VPlan.
- VFxUF.setUnderlyingValue(
- createStepForVF(Builder, TripCountV->getType(), State.VF, State.UF));
+ Value *RuntimeVF = nullptr;
+ if (VF.getNumUsers()) {
+ RuntimeVF = createStepForVF(Builder, TripCountV->getType(), State.VF, 1);
----------------
ayalz wrote:
>> could setting VF and/or VFxUF be done by optimizeForVFAndUF() instead, i.e., as soon as they are fixed for a VPlan?
> Could do, but may not be ideal to rely on an optimization to set them?
How about renaming the `optimizeForVFAndUF()` VPlanTransform to `setVFAndUF()`, or `setVF()` and `setUF()` VPlanTransforms, where fixing a constant value triggers its folding optimizations?
https://github.com/llvm/llvm-project/pull/95305
More information about the llvm-commits
mailing list