[PATCH] D115750: [SLP]Further improvement of the cost model for scalars used in buildvectors.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6450
+        [this, FTy, &Cost](ArrayRef<int> Mask,
+                           ArrayRef<const TreeEntry *> TEs) {
+          assert((TEs.size() == 1 || TEs.size() == 2) &&
----------------
It might make it cleaner if we pull out the lambda instead of embedding inside the performExtractsShuffleAction call?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115750