[llvm] ade2f10 - [SLP][NFCI] Don't trim indexes, reuse a variable (#149074)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 16 05:09:31 PDT 2025


Author: Piotr Fusik
Date: 2025-07-16T14:09:27+02:00
New Revision: ade2f1023d716bbf1473aaaf46d10faac73b014f

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

LOG: [SLP][NFCI] Don't trim indexes, reuse a variable (#149074)

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 6b307f7bac4f5..da6af353c709f 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -8622,11 +8622,10 @@ void BoUpSLP::tryToVectorizeGatheredLoads(
                                State == LoadsState::CompressVectorize)
                              return false;
                            ConsecutiveNodesSize += VL.size();
-                           unsigned Start = std::distance(Slice.begin(), It);
-                           unsigned Sz = Slice.size() - Start;
+                           size_t Start = std::distance(Slice.begin(), It);
+                           size_t Sz = Slice.size() - Start;
                            return Sz < VL.size() ||
-                                  Slice.slice(std::distance(Slice.begin(), It),
-                                              VL.size()) != VL;
+                                  Slice.slice(Start, VL.size()) != VL;
                          }))
                 continue;
               // Try to build long masked gather loads.


        


More information about the llvm-commits mailing list