dnsampaio requested changes to this revision. dnsampaio added a comment. This revision now requires changes to proceed.
`clang -### -target arm-arm-none-eabit -march=armv8-m.main+crypto` did not show +sha2 or +aes. After the patch it does. I believe that is not expected, as in ARM.td `crypto` is not applied for any M profile. And Arm®v8-M Architecture Reference Manual does not reference these extensions neither. ================ Comment at: lib/Driver/ToolChains/Arch/ARM.cpp:485 + + if (llvm::ARM::parseArchVersion(ArchSuffix) >= 8) { + auto CryptoIt = ---------------- ` && (llvm::ARM::parseArchProfile(Arch) == llvm::ARM::ProfileKind::A)` Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66018/new/ https://reviews.llvm.org/D66018 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits