[PATCH] D100486: [COST]Improve cost model for shuffles in SLP.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3620-3621
+            TargetTransformInfo::SK_PermuteSingleSrc;
+        if (ShuffleVectorInst::isReverseMask(NewMask))
+          SK = TargetTransformInfo::SK_Reverse;
+        CommonCost = TTI->getShuffleCost(SK, VecTy, NewMask);
----------------
Can the finding of a more specific ShuffleKind be done by getShuffleCost when a Mask is given?
It seems a bit inconvenient to have to do that manually before calling this function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100486