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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to