[PATCH] D113782: [SLP]Improve cost model for the shuffled extracts.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:5480
     Cost -= InsertCost;
+  } else if (FirstUsers.size() >= 2) {
+    unsigned MaxVF = *std::max_element(VF.begin(), VF.end());
----------------
RKSimon wrote:
> can FirstUsers be empty here ?
Yes, if no insertelement users (check line 5381) or insert index is non-constant (line 5384-5385)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113782