[llvm-branch-commits] [llvm] 3557e10 - Tweak comment

Benjamin Maxwell via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Feb 24 09:26:26 PST 2026


Author: Benjamin Maxwell
Date: 2026-02-24T17:25:07Z
New Revision: 3557e102aa2f39adc39470b4c90f18c9e73b8ffb

URL: https://github.com/llvm/llvm-project/commit/3557e102aa2f39adc39470b4c90f18c9e73b8ffb
DIFF: https://github.com/llvm/llvm-project/commit/3557e102aa2f39adc39470b4c90f18c9e73b8ffb.diff

LOG: Tweak comment

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
index 2a8bb4b53bcfa..d339385828e1c 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -5080,9 +5080,9 @@ void VPlanTransforms::materializeFactors(VPlan &Plan, VPBasicBlock *VectorPH,
   Type *TCTy = VPTypeAnalysis(Plan).inferScalarType(Plan.getTripCount());
   VPValue &VF = Plan.getVF();
   VPValue &VFxUF = Plan.getVFxUF();
-  // Note that after the transform, Plan.getVF and Plan.getVFxUF should not be
-  // used.
-  // TODO: Assert that they aren't used.
+  // Note that after the transform, no further uses of Plan.getVF and
+  // Plan.getVFxUF should be added.
+  // TODO: Add assertions for this.
 
   VPValue *UF =
       Plan.getOrAddLiveIn(ConstantInt::get(TCTy, Plan.getConcreteUF()));


        


More information about the llvm-branch-commits mailing list