On 07/05/15 17:24, James Greenhalgh wrote: > 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)
Committed as r223432 after fresh bootstrap and spec2k benchmarking. Thanks, Kugan