[PATCH] D92668: [SLP]Merge reorder and reuse shuffles.

[Anton Afanasyev via Phabricator via llvm-commits]

anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3513
+      } else if (!E->ReorderIndices.empty()) {
         DeadCost += TTI->getShuffleCost(
             TargetTransformInfo::SK_PermuteSingleSrc, VecTy);
----------------
So, you still can set `DeadCost = ...` instead of `+=`. It's simpler and points that we can get starting cost either from shuffle reuse, or from reorder indices, not both.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92668