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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 09:43:09 PDT 2022


RKSimon 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,
----------------
So CommonCost is only applied if (!MinBWs.count(VL0) || VecTy != SrcVecTy)  ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115757



More information about the llvm-commits mailing list