[PATCH] D47401: [X86] Rewrite the max and min reduction intrinsics to make better use of other functions and to reduce width to 256 and 128 bits were possible.

[Craig Topper via Phabricator via cfe-commits]

craig.topper created this revision.
craig.topper added reviewers: RKSimon, spatel, GBuella.

We only need to use 512 bit vectors all the way through v8i64 reductions since those max instructions are new to avx512f and only available in 512 bits until SKX.

For v16i32 and floating point we have legacy 128/256 bit instructions we can use.

I've tried to use other intrinsics to reduce the verbosity of the code and avoid having to mention all the shuffles. I've also removed all the -1 shuffle indices so the output sequence is fully specified and not left to backend optimization.


Repository:
  rC Clang

https://reviews.llvm.org/D47401

Files:
  lib/Headers/avx512fintrin.h
  test/CodeGen/avx512-reduceMinMaxIntrin.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47401.148694.patch
Type: text/x-patch
Size: 449334 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180526/31c68c81/attachment-0001.bin>