[NEON] Fix the architecture condition for the crypto intrinsics

[kosarev]

Summary

This is rather a cosmetic change as on ARMv7 targets we do not define __ARM_FEATURE_CRYPTO, even if it was explicitly requested with -target-feature. The crypto intrinsics test cases are in tools/clang/test/CodeGen/neon-crypto.c . No changes are necessary for them.

[SjoerdMeijer]

This looks correct to me, the ACLE indeed says that:

This is only available when __ARM_ARCH >= 8

Thanks for fixing this.

include/clang/Basic/arm_neon.td

 def FMINNMV : SInst<"vminnmv", "sd", "fQfQd">;
  
 ////////////////////////////////////////////////////////////////////////////////
 // Newly added Vector Extract for f64
 def VEXT_A64 : WInst<"vext", "dddi", "dQdPlQPl">;
 
 ////////////////////////////////////////////////////////////////////////////////
 // Crypto
-let ArchGuard = "__ARM_FEATURE_CRYPTO" in {
+let ArchGuard = "__ARM_ARCH >= 8 && defined(__ARM_FEATURE_CRYPTO)" in {
 def AESE : SInst<"vaese", "ddd", "QUc">;
 def AESD : SInst<"vaesd", "ddd", "QUc">;
 def AESMC : SInst<"vaesmc", "dd", "QUc">;
 def AESIMC : SInst<"vaesimc", "dd", "QUc">;
 
 def SHA1H : SInst<"vsha1h", "ss", "Ui">;
 def SHA1SU1 : SInst<"vsha1su1", "ddd", "QUi">;
 def SHA256SU0 : SInst<"vsha256su0", "ddd", "QUi">;