This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][AArch64] Add initial support for Cortex-A73


Hi James,

On 21/06/16 17:38, James Greenhalgh wrote:
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?


Ah, that was copied from the Cortex-a72 tuning. I didn't spend any time experimenting with it.
generic_branch_costs should be good enough for the initial enablement.
I can change it to cortexa57_branch_cost if you'd like.
Or we can do it separately later (I suspect Cortex-A72 should use those costs too.)



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!

Thanks, I'll rebase and commit it today.
Kyrill

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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]