[PATCH] D70068: [SLP] Enhance SLPVectorizer to vectorize vector aggregate

[Anton Afanasyev via Phabricator via llvm-commits]

anton-afanasyev marked 2 inline comments as done.
anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6769
+      int TmpUserCost;
+      SmallVector<Value *, 4> TmpBuildVectorOpds;
+      if (!findBuildVector(IE, TTI, TmpBuildVectorOpds, TmpUserCost))
----------------
anton-afanasyev wrote:
> dtemirbulatov wrote:
> > Maybe here we could try other sizes here, for example, 8?
> Are you talking about number 4 in `SmallVector<Value*, 4>`? It is just default size preallocated for vector, it could be potentially resized by `push_back()` below. But I believe it won't for most cases.
*default capacity


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70068/new/

https://reviews.llvm.org/D70068