On Thu, Jan 30, 2014 at 1:45 PM, Kyrill Tkachov <kyrylo.tkac...@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.

regards
Ramana

>
> Thanks,
> Kyrill
>
> 2014-01-30  Kyrylo Tkachov  <kyrylo.tkac...@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.

Reply via email to