This is the mail archive of the
mailing list for the GCC project.
Re: RFA: enable LRA for rs6000 [patch for WRF]
- From: Michael Meissner <meissner at linux dot vnet dot ibm dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, "Bergner, Peter" <bergner at vnet dot ibm dot com>, aavrunin at redhat dot com
- Date: Thu, 18 Apr 2013 17:05:44 -0400
- Subject: Re: RFA: enable LRA for rs6000 [patch for WRF]
- References: <5166F34C dot 30901 at redhat dot com> <20130415224853 dot GA17643 at ibm-tiger dot the-meissners dot org> <20130416225639 dot GA16621 at ibm-tiger dot the-meissners dot org> <516EAE5D dot 4080601 at redhat dot com> <20130417161042 dot GA22186 at ibm-tiger dot the-meissners dot org> <51705B25 dot 6020402 at redhat dot com>
On Thu, Apr 18, 2013 at 04:44:21PM -0400, Vladimir Makarov wrote:
> LRA can fix the wrong address but secondary reload was done before
> processing addresses. It could be fixed in rs6000.c code too but it
> is complicated and I found a better (and i think more right)
> solution by moving secondary reload generation after address
I tended to think secondary_reload should always happen after address
> Here is the patch for your branch (patch for trunk is a bit
> different as some changes in affected code were done on trunk).
> 2013-04-18 Vladimir Makarov <firstname.lastname@example.org>
> * lra-constraints.c (check_and_process_move): Move code for move
> cost check to simple_move_p. Remove equiv_substitution.
> (simple_move_p): New function.
> (curr_insn_transform): Use the new function. Move call of
> check_and_process_move after operand equiv substitution and
> address process.
> Tomorrow I am going to look at SPEC2006 dealII crash for 32-bit mode.
Thanks for the patch. Unfortunately I just updated the branch to be merged up
to 198065. So if you could send me a patch against current trunk, or update
the branch (branches/ibm/meissner-lra) and check it into the branch, it would
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460, USA
email: email@example.com, phone: +1 (978) 899-4797