sdesmalen added a comment.

In D75850#1912685 <https://reviews.llvm.org/D75850#1912685>, @simon_tatham 
wrote:

> > Renames the 'clang_arm_mve_alias' attribute into 'clang_arm_builtin_alias' 
> > (it will be used with CDE intrinsics as well as MVE intrinsics)
>
> You might talk to @sdesmalen, who mentioned introducing an `arm_sve_alias` 
> attribute in a comment on D75470 <https://reviews.llvm.org/D75470>. Probably 
> better if we all use the same one :-)


Thanks for pointing out! I added `arm_sve_alias` in D75861 
<https://reviews.llvm.org/D75861>, but am happy to update my patch to reuse the 
generic `__clang_arm_builtin_alias`!


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

https://reviews.llvm.org/D75850



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

Reply via email to