[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