[PATCH] D134681: [Clang][AArch64] Add SME outer product intrinsics
Richard Sandiford via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 14 09:40:17 PDT 2022
rsandifo-arm added a comment.
Thanks for the patch. This is going to be inconvenient, sorry, but: while implementing the specification in GCC, I noticed that the ZA functions weren't consistent about whether they had an `_m` suffix. `svwrite` (MOVA) had one, but the MOP intrinsics that you're implementing here didn't. Since SME2 does have some unpredicated instructions, it seems like it would be better to make the MOP intrinsics consistent with `svwrite`, with an `_m` suffix.
I've created https://github.com/ARM-software/acle/pull/218 for that change. Please let me know if it looks reasonable to you.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134681/new/
https://reviews.llvm.org/D134681
More information about the cfe-commits
mailing list