[PATCH] D43582: [SLP] Generalization of stores vectorization.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 5 10:08:51 PST 2018


lebedev.ri added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4598
   unsigned E = Stores.size();
-  IndexQueue.resize(E - 1);
+  const unsigned MaxLookDepth = std::max(E - 1, 16u);
+  SmallVector<unsigned, 16> IndexQueue(MaxLookDepth);
----------------
The `std::max()` is not a typo? Might warrant a comment.


Repository:
  rL LLVM

https://reviews.llvm.org/D43582





More information about the llvm-commits mailing list