[llvm] [SLP] Use named structs in vectorizeStores() (NFC) (PR #132781)

[Alexey Bataev via llvm-commits]

================
@@ -19923,31 +19923,51 @@ bool SLPVectorizerPass::vectorizeStores(
   BoUpSLP::ValueSet VectorizedStores;
   bool Changed = false;
 
+  /// A store instruction and the distance of its address to a base pointer.
+  struct CandidateInstr {
+    CandidateInstr(unsigned InstrIdx, int DistToBasePtr)
+        : InstrIdx(InstrIdx), DistToBasePtr(DistToBasePtr) {}
+    unsigned InstrIdx;
+    int DistToBasePtr;
+  };
   struct StoreDistCompare {
-    bool operator()(const std::pair<unsigned, int> &Op1,
-                    const std::pair<unsigned, int> &Op2) const {
-      return Op1.second < Op2.second;
+    bool operator()(const CandidateInstr &Op1,
+                    const CandidateInstr &Op2) const {
+      return Op1.DistToBasePtr < Op2.DistToBasePtr;
     }
   };
-  // A set of pairs (index of store in Stores array ref, Distance of the store
-  // address relative to base store address in units).
-  using StoreIndexToDistSet =
-      std::set<std::pair<unsigned, int>, StoreDistCompare>;
-  auto TryToVectorize = [&](const StoreIndexToDistSet &Set) {
+
+  /// A group of store instructions that we'll try to bundle together.
+  /// They are ordered using their signed distance to the address of this
+  /// set's BaseInstr.
+  struct CandidateBundle {
----------------
alexey-bataev wrote:

`Bundle` is reserved for scheduling model. Better to use something like `StoresIndicesDistsSet` or keep the original name.
Also, better to move these structs out of function into the anonymous namespace

https://github.com/llvm/llvm-project/pull/132781