This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] reload: Try alternative with swapped operands before going to the next
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Jeff Law <law at redhat dot com>, Maxim Kuvyrkov <maxim at codesourcery dot com>, Richard Kenner <kenner at vlsi1 dot ultra dot nyu dot edu>
- Date: Thu, 17 Nov 2011 17:48:09 +0100
- Subject: Re: [PATCH] reload: Try alternative with swapped operands before going to the next
- References: <20111114172108.GA8784@bart> <4EC34FCD.3070808@redhat.com> <4EC4FFD6.7000807@linux.vnet.ibm.com>
> I understand your concern and the patch indeed will have an impact on the
> generated code of probably several back-ends. But with entering stage 3 the
> backend maintainers anyway are supposed to have a closer look at
> regressions.
Yes, that's why risking to introduce more of them isn't the best thing to do at
this stage. IMO this is too late in the 4.7 cycle.
> In my opinion the patch makes reload behave more like everybody expects it to
> behave and at least for s390 I can say that we have several patterns which
> have been written with that expectation. With the patch GCC generates better
> code for S/390 so I would like to see this patch applied.
That isn't compelling enough I think, given the timing.
--
Eric Botcazou