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

Reply via email to