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