> Marcus Shawcroft wrote: > On 5 March 2015 at 14:49, Wilco Dijkstra <wdijk...@arm.com> wrote: > > This patch fixes the shift costs for Cortex-A53 so they are more accurate - > > immediate shifts > use > > SBFM/UBFM which takes 2 cycles, register controlled shifts take 1 cycle. > > Bootstrap and > regression > > OK. > > > > ChangeLog: > > 2015-03-05 Wilco Dijkstra <wdijk...@arm.com> > > > > * gcc/config/arm/aarch-cost-tables.h (cortexa53_extra_costs): > > Make Cortex-A53 shift costs more accurate. > > OK /Marcus
Kyrill, could you check this in please? Patch attached Wilco 2015-05-01 Wilco Dijkstra <wdijk...@arm.com> * gcc/config/arm/aarch-cost-tables.h (cortexa53_extra_costs): Make Cortex-A53 shift costs more accurate.
--- gcc/config/arm/aarch-cost-tables.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/gcc/config/arm/aarch-cost-tables.h b/gcc/config/arm/aarch-cost-tables.h index 05e96a9..6bb8ede 100644 --- a/gcc/config/arm/aarch-cost-tables.h +++ b/gcc/config/arm/aarch-cost-tables.h @@ -130,12 +130,12 @@ const struct cpu_cost_table cortexa53_extra_costs = 0, /* arith. */ 0, /* logical. */ COSTS_N_INSNS (1), /* shift. */ - COSTS_N_INSNS (2), /* shift_reg. */ + 0, /* shift_reg. */ COSTS_N_INSNS (1), /* arith_shift. */ - COSTS_N_INSNS (2), /* arith_shift_reg. */ + COSTS_N_INSNS (1), /* arith_shift_reg. */ COSTS_N_INSNS (1), /* log_shift. */ - COSTS_N_INSNS (2), /* log_shift_reg. */ - 0, /* extend. */ + COSTS_N_INSNS (1), /* log_shift_reg. */ + COSTS_N_INSNS (1), /* extend. */ COSTS_N_INSNS (1), /* extend_arith. */ COSTS_N_INSNS (1), /* bfi. */ COSTS_N_INSNS (1), /* bfx. */ -- 1.9.1