On Wed, Mar 19, 2014 at 9:55 AM, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote:
> Hi all,
>
> In order to properly cost the rev16 instruction we need a new field in the
> cost tables.
> This patch adds that and specifies its value for the existing cost tables.
> Since rev16 is used to implement the BSWAP operation we add handling of that
> in the rtx cost function using the new field.
>
> Tested on arm-none-eabi and bootstrapped on an arm linux target.
>
> Does it look ok for stage1?

Ok for stage1 if no regressions.

>
> Thanks,
> Kyrill
>
> 2014-03-19  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
>
>     * config/arm/aarch-common-protos.h (alu_cost_table): Add rev field.
>     * config/arm/aarch-cost-tables.h (generic_extra_costs): Specify
>     rev cost.
>     (cortex_a53_extra_costs): Likewise.
>     (cortex_a57_extra_costs): Likewise.
>     * config/arm/arm.c (cortexa9_extra_costs): Likewise.
>     (cortexa7_extra_costs): Likewise.
>     (cortexa12_extra_costs): Likewise.
>     (cortexa15_extra_costs): Likewise.
>     (v7m_extra_costs): Likewise.
>     (arm_new_rtx_costs): Handle BSWAP.

Reply via email to