[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