[llvm] 9fce1fc - [LVP] Fix comment for hasPlanWithVF (NFC).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 30 11:14:01 PDT 2023


Author: Florian Hahn
Date: 2023-04-30T19:13:53+01:00
New Revision: 9fce1fc6f8344871b868aa250c0397e76ed40e91

URL: https://github.com/llvm/llvm-project/commit/9fce1fc6f8344871b868aa250c0397e76ed40e91
DIFF: https://github.com/llvm/llvm-project/commit/9fce1fc6f8344871b868aa250c0397e76ed40e91.diff

LOG: [LVP] Fix comment for hasPlanWithVF (NFC).

The function checks if there's a plan with the specified VF. Update the
comment to match the implementation.

Pointed out as independent improvement in D143938.

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
index 317e31107ce22..3a35f3b754743 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
@@ -319,8 +319,8 @@ class LoopVectorizationPlanner {
   void printPlans(raw_ostream &O);
 #endif
 
-  /// Look through the existing plans and return true if we have one with all
-  /// the vectorization factors in question.
+  /// Look through the existing plans and return true if we have one with
+  /// vectorization factor \p VF.
   bool hasPlanWithVF(ElementCount VF) const {
     return any_of(VPlans,
                   [&](const VPlanPtr &Plan) { return Plan->hasVF(VF); });


        


More information about the llvm-commits mailing list