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
