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.tkac...@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.
