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

[Dinar Temirbulatov via Phabricator via llvm-commits]

dtemirbulatov 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:
> 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
yes, correct


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70068