This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM] Add Cortex-A57 rtx costs table
- From: Jakub Jelinek <jakub at redhat dot com>
- To: ramrad01 at arm dot com
- Cc: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, Richard Biener <rguenther at suse dot de>
- Date: Thu, 6 Feb 2014 17:39:41 +0100
- Subject: Re: [PATCH][ARM] Add Cortex-A57 rtx costs table
- Authentication-results: sourceware.org; auth=none
- References: <52EA5764 dot 8000607 at arm dot com> <CAJA7tRZf0ZcTq9-8-0YnHGyjEsPn-K7NrLJhX-V1oEtfJTmd=g at mail dot gmail dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Feb 06, 2014 at 04:37:38PM +0000, Ramana Radhakrishnan wrote:
> On Thu, Jan 30, 2014 at 1:45 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> > Hi all,
> >
> > This patch adds the rtx costs table for Cortex-A57 and sets its issue rate
> > properly in the arm backend.
> >
> > Tested on arm-none-eabi on a model.
> >
> >
> > Ok for trunk?
>
> In my view this is OK - this is just a tuning patch for A57 and we
> should be able to take it now as it is not default anywhere and
> unlikely to affect anyone doing bulk rebuilds.
>
> However I would like an RM to ack this.
Ok.
> > 2014-01-30 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
> >
> > * config/arm/aarch-cost-tables.h (cortexa57_extra_costs): New table.
> > Remove extra newline at end of file.
> > * config/arm/arm.c (arm_cortex_a57_tune): New tuning struct.
> > (arm_issue_rate): Handle cortexa57.
> > * config/arm/arm-cores.def (cortex-a57): Use cortex_a57 tuning.
> > (cortex-a57.cortex-a53): Likewise.
Jakub