[PATCH] D90613: [x86] add AVX2 cost model entries for maxnum of 256-bit vectors

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 07:46:28 PST 2020


spatel created this revision.
spatel added reviewers: RKSimon, pengfei, craig.topper.
Herald added subscribers: hiraditya, mcrosier.
Herald added a project: LLVM.
spatel requested review of this revision.

As noticed in D90554 <https://reviews.llvm.org/D90554> , the AVX2 costs for 256-bit vectors did not include FMAXNUM entries, so we fell back to AVX1 which assumes those ops will be split into 128-bit halves.


https://reviews.llvm.org/D90613

Files:
  llvm/lib/Target/X86/X86TargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/X86/fmaxnum.ll
  llvm/test/Analysis/CostModel/X86/fminnum.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90613.302270.patch
Type: text/x-patch
Size: 20155 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201102/447cac1d/attachment.bin>


More information about the llvm-commits mailing list