On 17/11/14 16:59, Ramana Radhakrishnan wrote:
On Mon, Nov 17, 2014 at 2:48 PM, Kyrill Tkachov <kyrylo.tkac...@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
Makes sense.
Here's what I propose.
Ok?
Kyrill
Ramana
Thanks,
Kyrill
2014-11-17 Kyrylo Tkachov <kyrylo.tkac...@arm.com>
* config/aarch64/aarch64-cores.def (cortex-a53): Remove
AARCH64_FL_CRYPTO from feature flags.
(cortex-a57): Likewise.
(cortex-a57.cortex-a53): Likewise.
Index: htdocs/gcc-5/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.23
diff -U 3 -r1.23 changes.html
--- htdocs/gcc-5/changes.html 14 Nov 2014 10:49:51 -0000 1.23
+++ htdocs/gcc-5/changes.html 17 Nov 2014 17:46:41 -0000
@@ -257,6 +257,13 @@
Alternatively it can be enabled by default by configuring GCC with the
<code>--enable-fix-cortex-a53-835769</code> option.
</li>
+ <li> The cryptographic extensions to the ARMv8-A architecture are no
+ longer enabled by default when specifying the
+ <code>-mcpu=cortex-a53</code>, <code>-mcpu=cortex-a57</code> or
+ <code>-mcpu=cortex-a57.cortex-a53</code> options. To enable these
+ extensions add the <code>+crypto</code> extension to your given
+ <code>-mcpu</code> or <code>-march</code> options' value.
+ </li>
</ul>
