This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM][1/3] Add rev field to rtx cost tables
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Fri, 28 Mar 2014 14:20:06 +0000
- Subject: Re: [PATCH][ARM][1/3] Add rev field to rtx cost tables
- Authentication-results: sourceware.org; auth=none
- References: <53296995 dot 7050009 at arm dot com>
- Reply-to: ramrad01 at arm dot com
On Wed, Mar 19, 2014 at 9:55 AM, Kyrill Tkachov <kyrylo.tkachov@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.tkachov@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.