[AArch64][PR65375] Fix RTX cost for vector SET
Kugan
kugan.vivekanandarajah@linaro.org
Wed May 20 03:32:00 GMT 2015
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 <kuganv@linaro.org>
>>>> Jim Wilson <jim.wilson@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
More information about the Gcc-patches
mailing list