[all-commits] [llvm/llvm-project] 3d59f9: [ARM][AArch64] Correct __ARM_FEATURE_CRYPTO macro ...

[dcandler via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3d59f9d22440645ca0237dfc5d91ca09f749174b
      https://github.com/llvm/llvm-project/commit/3d59f9d22440645ca0237dfc5d91ca09f749174b
  Author: David Candler <david.candler at arm.com>
  Date:   2021-05-14 (Fri, 14 May 2021)

  Changed paths:
    M clang/lib/Basic/Targets/AArch64.cpp
    M clang/lib/Basic/Targets/ARM.cpp
    M clang/lib/Driver/ToolChains/Arch/ARM.cpp

  Log Message:
  -----------
  [ARM][AArch64] Correct __ARM_FEATURE_CRYPTO macro and crypto feature

This patch contains a couple of minor corrections to my previous
crypto patch:

Since both AArch32 and AArch64 are now correctly setting the aes and
sha2 features individually, it is not necessary to continue to check
the crypto feature when defining feature macros.

In the AArch32 driver, the feature vector is only modified when the
crypto feature is actually in the vector. If crypto is not present,
there is no need to split it and explicitly define crypto/sha2/aes.

Reviewed By: lenary

Differential Revision: https://reviews.llvm.org/D102406