[PATCH] D55452: [CostModel][X86] Teach getArithmeticReductionCost to properly cost the shuffles needed for a PairWise reduction before we reach a legal type. And remove one shuffle from the end of the reduction.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 8 17:43:38 PST 2018


craig.topper updated this revision to Diff 177402.
craig.topper added a comment.

Use ShufLT.second.getVectorNumElements() to create the ShufTy.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55452

Files:
  include/llvm/CodeGen/BasicTTIImpl.h
  test/Analysis/CostModel/X86/reduction.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55452.177402.patch
Type: text/x-patch
Size: 28788 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181209/3fa31d1c/attachment.bin>


More information about the llvm-commits mailing list