stuij marked an inline comment as done.
stuij added inline comments.
================
Comment at: llvm/include/llvm/Support/ARMTargetParser.def:127
+ ARM::AEK_HWDIVTHUMB | ARM::AEK_DSP | ARM::AEK_CRC | ARM::AEK_RAS |
+ ARM::AEK_DOTPROD | ARM::AEK_BF16 | ARM::AEK_I8MM))
ARM_ARCH("armv9-a", ARMV9A, "9-A", "v9a",
----------------
tmatheson wrote:
> No ARM::AEK_SHA2 | ARM::AEK_AES? Or does 8.8 need updated?
Yes, I think 8.8 needs update, and some other arches as well.
In the A profile armarm, section A2.3, it is stated that from 8.2 SME(2) and
EAS aren't by default included in the cryptographic extension as the
Cryptographic Extension in an implementation is subject to export license
controls. Inclusion of the extension can be either/or or none, so we should
default to none.
I think this should be handled by separate patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138010/new/
https://reviews.llvm.org/D138010
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
