[PATCH] D75850: [ARM, CDE] Generalize MVE intrinsics infrastructure to support CDE

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 9 10:14:14 PDT 2020


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





More information about the llvm-commits mailing list