[PATCH] D91994: [AArch64] Cortex-R82: remove crypto

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer updated this revision to Diff 309216.
SjoerdMeijer added a comment.

The short story is that this was "accidentally" getting things right.

The long story story, is that after discussing this offline, we want to this to be more aligned with GCC which enables a more minimal set of architecture extensions. This means that we don't always want to enable optional extensions, like crypto in this case. Additionally, none of the R-profile cores implement crypto, so enabling it might not be the best default for R-profile.


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

https://reviews.llvm.org/D91994

Files:
  clang/lib/Driver/ToolChains/Arch/AArch64.cpp
  clang/test/Preprocessor/aarch64-target-features.c
  llvm/include/llvm/Support/AArch64TargetParser.def
  llvm/unittests/Support/TargetParserTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91994.309216.patch
Type: text/x-patch
Size: 9074 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201203/b28a29e8/attachment.bin>