[PATCH] D28118: AVX-512 cost calculation for interleave load/store patterns

[Elena Demikhovsky via Phabricator via llvm-commits]

delena added inline comments.


================
Comment at: ../lib/Analysis/CostModel.cpp:115
+
+static bool isBroadcastVectorMask(SmallVectorImpl<int> &Mask) {
+  int SplatVal = -1;
----------------
RKSimon wrote:
> I've added a ShuffleVectorInst::isSplat helper in D27811 for the same purpose. It's probably worth you merging that part (inc the CodeGenPrepare.cpp change as well) here.
The "broadcast" was not the matter of my patch and I do not really investigated in it, just added for completeness. And the broadcast-test is not full. I'd rather remove these changes at all and let you to proceed. 


Repository:
  rL LLVM

https://reviews.llvm.org/D28118