[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.

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 26 14:06:59 PDT 2018


RKSimon added inline comments.


================
Comment at: cfe/trunk/lib/Headers/avx512fintrin.h:9855
+  __v8di __t6 = (__v8di)_mm512_##op(__t4, __t5); \
+  return __t6[0];
 
----------------
Would it be dumb to allow VLX capable CPUs to use 128/256 variants of the VPMAXUQ etc ? Or is it better to focus on improving SimplifyDemandedElts to handle this (and many other reduction cases that all tend to keep to the original vector width)?


Repository:
  rL LLVM

https://reviews.llvm.org/D47401





More information about the cfe-commits mailing list