[PATCH] D27811: [CostModel][X86] Add support for broadcast shuffle costs

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 10:59:13 PST 2017


andreadb added inline comments.


================
Comment at: lib/Analysis/CostModel.cpp:520-522
+      if (Shuffle->isSplat(&BroadcastIndex) && BroadcastIndex == 0)
+        return TTI->getShuffleCost(TargetTransformInfo::SK_Broadcast,
+                                   VecTypOp0, 0, nullptr);
----------------
Is this code still needed?
r290810 introduced a check for `isZeroEltBroadcastVectorMask' at line 530.


Repository:
  rL LLVM

https://reviews.llvm.org/D27811





More information about the llvm-commits mailing list