This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Marcus Shawcroft <marcus dot shawcroft at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Mon, 17 Nov 2014 16:59:33 +0000
- Subject: Re: [PATCH][AArch64] Remove crypto extension from default for cortex-a53, cortex-a57
- Authentication-results: sourceware.org; auth=none
- References: <546A0AC1 dot 9060909 at arm dot com>
- Reply-to: ramrad01 at arm dot com
On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi all,
>
> Some configurations of Cortex-A53 and Cortex-A57 don't ship with crypto,
> so enabling it by default for -mcpu=cortex-a53 and cortex-a57 is
> inappropriate.
>
> Tested aarch64-none-elf. Reminder that at the moment all the crypto
> extension does is enable the use of the ACLE crypto intrinsics in arm_neon.h
>
> Ok for trunk?
I can't ok this but ...
Since we've changed behaviour from 4.9 I think it warrants an entry in
changes.html for 5.0
Ramana
>
> Thanks,
> Kyrill
>
> 2014-11-17 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * config/aarch64/aarch64-cores.def (cortex-a53): Remove
> AARCH64_FL_CRYPTO from feature flags.
> (cortex-a57): Likewise.
> (cortex-a57.cortex-a53): Likewise.