This revision was automatically updated to reflect the committed changes.
Closed by commit rL330187: [NEON] Fix the architecture condition for the crypto
intrinsics (authored by kosarev, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D
This revision was automatically updated to reflect the committed changes.
Closed by commit rC330187: [NEON] Fix the architecture condition for the crypto
intrinsics (authored by kosarev, committed by ).
Repository:
rL LLVM
https://reviews.llvm.org/D45669
Files:
include/clang/Basic/arm_neon.
SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.
This looks correct to me, the ACLE indeed says that:
> This is only available when __ARM_ARCH >= 8
Thanks for fixing this.
https://reviews.llvm.org/D45669
___
kosarev created this revision.
kosarev added reviewers: t.p.northover, rengolin, SjoerdMeijer.
kosarev added a project: clang.
Herald added subscribers: kristof.beyls, javed.absar.
This is rather a cosmetic change as on ARMv7 targets we do not define
__ARM_FEATURE_CRYPTO, even if it was explicitl