This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][AArch64] Add initial support for Cortex-A73
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at foss dot 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>, <nd at arm dot com>
- Date: Tue, 21 Jun 2016 17:38:19 +0100
- Subject: Re: [PATCH][AArch64] Add initial support for Cortex-A73
- Authentication-results: sourceware.org; auth=none
- Nodisclaimer: True
- References: <5755A250 dot 9050609 at foss dot arm dot com> <5769637F dot 8090603 at foss dot arm dot com>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
On Tue, Jun 21, 2016 at 04:55:43PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This is a rebase of https://gcc.gnu.org/ml/gcc-patches/2016-06/msg00403.html
> on top of Evandro's changes.
> Also, to elaborate on the original posting, the initial tuning structure is
> based on the Cortex-A57 one but with the issue rate set to 2, FMA steering
> turned off and ADRP+LDR fusion enabled.
I see you've also chosen to use the generic_branch_cost costs for
branches. As you didn't mention it explicitly here, was that intentional?
> Is this ok for trunk?
This looks OK to me. Watch out for the conflict with the Broadcom Vulcan
patch that was committed to trunk earlier today. The merge should be easy.
Thanks for the patch!
James
> 2016-06-21 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * config/aarch64/aarch64.c (cortexa73_tunings): New struct.
> * config/aarch64/aarch64-cores.def (cortex-a73): New entry.
> (cortex-a73.cortex-a35): Likewise.
> (cortex-a73.cortex-a53): Likewise.
> * config/aarch64/aarch64-tune.md: Regenerate.
> * doc/invoke.texi (AArch64 Options): Document cortex-a73,
> cortex-a73.cortex-a35 and cortex-a73.cortex-a53 arguments to
> -mcpu and -mtune.