[PATCH] D101555: [SLP]Improve handling of compensate external uses cost.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:2901
+        auto *I = cast<Instruction>(V);
+        if (!SourceVectors.contains(I)) {
+          ++NotUsedVectors;
----------------
would this be better as a count_if ?


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:2921
 
-        TE->setOperand(1, VectorOperands);
+      ValueList VectorOperands;
+      for (Value *V : VL)
----------------
Any good way to merge the SourceVectors set with the VectorOperands list?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101555