[PATCH] D115800: [SLP]Do not represent splats as node with the reused scalars.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4739
       // broadcast.
-      return TTI->getShuffleCost(TargetTransformInfo::SK_Broadcast, VecTy);
+      return TTI->getShuffleCost(TargetTransformInfo::SK_Broadcast, FinalVecTy);
     }
----------------
RKSimon wrote:
> Is this a separate fix?
Same, but actually there should not be a difference between VecTy and FinalVecTy. Probably, better to add the assert here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115800