On Wed, May 06, 2015 at 03:12:33AM +0100, Kugan wrote: > >> gcc/ChangeLog: > >> > >> 2015-04-24 Kugan Vivekanandarajah <kug...@linaro.org> > >> Jim Wilson <jim.wil...@linaro.org> > >> > >> * config/arm/aarch-common-protos.h (struct mem_cost_table): Added > >> new fields loadv and storev. > >> * config/aarch64/aarch64-cost-tables.h (thunderx_extra_costs): > >> Initialize loadv and storev. > >> * config/arm/aarch-cost-tables.h (generic_extra_costs): Likewise. > >> (cortexa53_extra_costs): Likewise. > >> (cortexa57_extra_costs): Likewise. > >> (xgene1_extra_costs): Likewise. > >> * config/aarch64/aarch64.c (aarch64_rtx_costs): Update vector > >> rtx_costs. > > Thanks James for the review. Attached patch changes this. Is this OK ?
Hi Kugan, Thanks for sticking with it through a long review, sorry that the replies have been patchy, I'm still travelling. This patch is OK for trunk, with an updated ChangeLog and assuming no regressions after a test run (And a quick check with some popular benchmarks if possible) Thanks, and sorry again for the delay, James
