[llvm] 0a95a86 - [VPlan] Fix first-lane comment in sinkScalarOperands (NFC) (#166347)

via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 4 04:03:02 PST 2025


Author: Ramkumar Ramachandra
Date: 2025-11-04T12:02:58Z
New Revision: 0a95a86634a8a8d2e95a390828e345058b4a723f

URL: https://github.com/llvm/llvm-project/commit/0a95a86634a8a8d2e95a390828e345058b4a723f
DIFF: https://github.com/llvm/llvm-project/commit/0a95a86634a8a8d2e95a390828e345058b4a723f.diff

LOG: [VPlan] Fix first-lane comment in sinkScalarOperands (NFC) (#166347)

To follow-up on a post-commit review.

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 e8e87aa8a4f3c..2588c878d8472 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -198,8 +198,8 @@ static bool sinkScalarOperands(VPlan &Plan) {
     VPSingleDefRecipe *SinkCandidate;
     std::tie(SinkTo, SinkCandidate) = WorkList[I];
 
-    // All recipe users of the sink candidate must be in the same block SinkTo
-    // or all users outside of SinkTo must have only their first lane used. In
+    // All recipe users of SinkCandidate must be in the same block SinkTo or all
+    // users outside of SinkTo must only use the first lane of SinkCandidate. In
     // the latter case, we need to duplicate SinkCandidate.
     auto UsersOutsideSinkTo =
         make_filter_range(SinkCandidate->users(), [SinkTo](VPUser *U) {


        


More information about the llvm-commits mailing list