[PATCH] D115757: [SLP]Generalize cost model.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6552
+            if (!MinBWs.count(VL0) || VecTy != SrcVecTy) {
+              VecCost = CommonCost +
+                        TTI->getCastInstrCost(E->getOpcode(), VecTy, SrcVecTy,
----------------
RKSimon wrote:
> So CommonCost is only applied if (!MinBWs.count(VL0) || VecTy != SrcVecTy)  ?
Yeah, looks like should not ignore it here, since it includes the shuffle cost. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115757