Re: [PATCH] [AArch64] Distinct costs for sign and zero extension

[James Greenhalgh <james.greenhalgh@arm.com>]

On Tue, Oct 20, 2015 at 07:52:45AM +0800, Andrew Pinski wrote:
> On Tue, Oct 20, 2015 at 7:10 AM, Evandro Menezes <e.menezes@samsung.com> wrote:
> > Some micro-architectures may favor one of sign or zero extension over the
> > other in the base plus extended register offset addressing mode.
> 
> Yes I was going to create the same patch as ThunderX is one of those
> micro-architectures.

OK (Though the patch would have looked better had all the in-tree examples
not be no-op splits!).

I've committed this as r229431 on your behalf. Please provide full ChangeLog
entries with the patch submission, I derived the Name/Email fields and
committed this:

2015-10-27  Evandro Menezes  <e.menezes@samsung.com>

	* config/aarch64/aarch64-protos.h (cpu_addrcost_table): Split member
	for register extension into sign and zero register extension.
	* config/aarch64/aarch64.c (generic_addrcost_table): Infer values
	for sign and zero register extension.
	(cortexa57_addrcost_table): Likewise.
	(xgene1_addrcost_table): Likewise.

Thanks,
James

> >
> > This patch separates the member "register_extend" of the structure
> > "cpu_addrcost_table" into two, one for sign and the other zero extension.
> >
> > Please, commit if it's alright.
> >
> > Thank you,
> >
> > --
> > Evandro Menezes
> >
>