FreddyYe added a comment.

In D112777#3114502 <https://reviews.llvm.org/D112777#3114502>, @craig.topper 
wrote:

> Not directly related to this patch, but why is the suffix _pch and _sch when 
> the instruction names end in CPH and CSH? It kind of seems like the correct 
> intrinsic name would have been _mm_fmulc_ph.
>
> Why does the name here need to be aligned with mul_ps/pd? This a "complex" 
> multiply which is a different operation. Is gcc also going to add aliases?

I can answer the second question. The prefix "f" can be judged as the mnemonic 
to distinguish fma instrinsics. The suffix "c" can be judged as the mnemonic of 
"complex". So add "f" mnemonic in this multiply intrinsics is ambiguous. gcc 
will add aliases, too. The first question is also a good question. But for now, 
it's not very conflict to old intrinsics, I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112777

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

Reply via email to