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: Gerald Pfeifer <gerald at pfeifer dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, 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: Tue, 25 Nov 2014 02:36:38 +0100 (CET)
- 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> <CAJA7tRapKphoSY=TizZoKc5UEw-OybVCOQwxZM5nGkmenDFK0g at mail dot gmail dot com> <546B13A4 dot 9070204 at arm dot com>
On Tuesday 2014-11-18 09:38, Kyrill Tkachov wrote:
Here's what I propose.
+ <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.
"option's"?
Or better "to the value of your...option(s)"?
The description talks about -mcpu and mentions -march only once.
Isn't this a bit confusing?
Gerald
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>