[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 22 18:57:59 PDT 2021


pengfei added a comment.

By the way, we synced with GCC and we are using the same order in the builtins now. https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=db3b96df03fdbe2fb770729501e2e9b65e66c2da;hp=ed643e9f171e99b0aa1453b3f29ed1103e9b5c80
We still have some different builtin names due to historical reasons, e.g. `__builtin_ia32_vfmulcsh_mask_round` (gcc), `__builtin_ia32_vfcmulcsh_mask` (llvm). I think it's OK for now.
Can we let this patch in? I will solve the multi evaluation problem in another patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109658/new/

https://reviews.llvm.org/D109658



More information about the llvm-commits mailing list