[PATCH] D141606: [AArch64] Remove AES, SHA2, SHA3 and SM4 features from armv8.6-a+

[Philipp Tomsich via Phabricator via llvm-commits]

philipp.tomsich added a comment.

I reread the ARMv8-A architecture today and am a little confused now: it looks as if the crypto features (i.e., FEAT_AES and friends) are always optional.
This is not surprising as the the section "The ARMv8 Cryptographic Extension" states that "[t]he presence of the Cryptographic Extension in an implementation is subject to export license controls."

So should we remove these from the defaults for all architecture-levels and include them in the relevant cores only?
Or am I reading the architecture wrong?


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

https://reviews.llvm.org/D141606