[PATCH] D96381: [AArch64] Adding SHA3 Intrinsics support

Ana Pazos via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 16 10:46:00 PST 2021


apazos added a comment.

This looks like a straightforward implementation. The only caveat is that the XAR immediate does not represent a lane, and hence the need for a custom immediate range check. Looks sensible to me.
@labrinea   and others at ARM, do have any other comment before this is merged?


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

https://reviews.llvm.org/D96381



More information about the cfe-commits mailing list