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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 29 02:56:04 PST 2016


RKSimon added a comment.

https://reviews.llvm.org/D27811 is working on much the same code, but I'm fine with you getting these changes in first and I'll carry on the refactor afterwards.



================
Comment at: ../lib/Analysis/CostModel.cpp:115
+
+static bool isBroadcastVectorMask(SmallVectorImpl<int> &Mask) {
+  int SplatVal = -1;
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D28118





More information about the llvm-commits mailing list