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.mene...@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.mene...@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 > > >