Followup to D87604, having confirmed on PR47506 that we can use the llvm codegen expansion for fadd/fmul as well.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.
| clang/lib/Headers/avx512fintrin.h | ||
|---|---|---|
| 9559 | Better to change min and max as well. | |
| clang/lib/Headers/avx512fintrin.h | ||
|---|---|---|
| 9559 | OK - I'll do that as a followup immediately after this patch - thanks! | |
Better to change min and max as well.