[PATCH 4/4] AArch64: Add regmove_costs for Cortex-A57 and A53

Wilco Dijkstra wdijkstr@arm.com
Thu Sep 11 15:15:00 GMT 2014


I've kept the integer move costs at 1 - patch attached for commit as I don't have write access.

ChangeLog:
2014-09-11  Wilco Dijkstra  <wdijkstr@arm.com>

	* gcc/config/aarch64/aarch64.c:
	(cortexa57_regmove_cost): New cost table for A57. 
	(cortexa53_regmove_cost): New cost table for A53.
	Increase GP2FP/FP2GP cost to spilling from integer to FP registers.

> -----Original Message-----
> From: Marcus Shawcroft [mailto:marcus.shawcroft@gmail.com]
> Sent: 04 September 2014 17:40
> To: Wilco Dijkstra
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH 4/4] AArch64: Add regmove_costs for Cortex-A57 and A53
> 
> On 4 September 2014 15:47, Wilco Dijkstra <wdijkstr@arm.com> wrote:
> > This patch adds regmove_costs for Cortex-A57 and A53, and sets the cost of GP2FP/FP2GP
> higher than
> > memory cost to block the register allocator allocating integer values in FP registers.
> >
> > Overall these patches give 2-3% speedup on SPEC.
> >
> > This passes all regression tests (with this fix
> > https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00356.html).
> >
> > OK for commit?
> >
> > Wilco
> >
> > ChangeLog:
> > 2014-09-04  Wilco Dijkstra  <wdijkstr@arm.com>
> >
> >         * gcc/config/aarch64/aarch64.c:
> >         Add cortexa57_regmove_cost and cortexa53_regmove_cost to avoid
> >         spilling from integer to FP registers.
> 
> Write a proper ChangeLog entry please.
> 
> Keep the GP2GP cost aligned with generic until we have justification
> to change it.
> 
> /Marcus
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: Add-regmove_cost-for-A57-and-A53.txt
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140911/dcf720a0/attachment.txt>


More information about the Gcc-patches mailing list