[llvm] 75b8f98 - [SLP] NFC. Change the comment to match the code execution. (#116022)

via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 20:42:23 PST 2024


Author: Han-Kuan Chen
Date: 2024-11-21T12:42:20+08:00
New Revision: 75b8f98ef69cc43289af4bddfa04e1cf90cc3d86

URL: https://github.com/llvm/llvm-project/commit/75b8f98ef69cc43289af4bddfa04e1cf90cc3d86
DIFF: https://github.com/llvm/llvm-project/commit/75b8f98ef69cc43289af4bddfa04e1cf90cc3d86.diff

LOG: [SLP] NFC. Change the comment to match the code execution. (#116022)

Make code execute like the comment will modify many tests and affect the
performance. As a result, we change the comment instead of the code.

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
index 47dcde7d9d1899..4b661ad40f2d46 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -2222,7 +2222,7 @@ class BoUpSLP {
       MapVector<unsigned, std::pair<unsigned, unsigned>> HashMap;
       // Try to be closer to the original results, if we have multiple lanes
       // with same cost. If 2 lanes have the same cost, use the one with the
-      // lowest index.
+      // highest index.
       for (int I = getNumLanes(); I > 0; --I) {
         unsigned Lane = I - 1;
         OperandsOrderData NumFreeOpsHash =


        


More information about the llvm-commits mailing list