pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks for bringing this refactor.
I also verified that ICC and GCC both do reduce math in an binary tree way,
though sometimes ICC has a different LSB from GCC and Clang.
================
Comment at: clang/lib/Headers/avx512fintrin.h:9559
static __inline__ double __DEFAULT_FN_ATTRS512
_mm512_reduce_max_pd(__m512d __V) {
_mm512_mask_reduce_operator(max_pd);
----------------
Better to change min and max as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92940/new/
https://reviews.llvm.org/D92940
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
