This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH] m68k: update various structures for arg pointer
On Wed, Mar 10, 2004 at 12:10:56PM -0800, Richard Henderson wrote:
> On Wed, Mar 10, 2004 at 11:24:36AM +0100, Gunther Nikl wrote:
> > I wonder if this patch will make it into the 3.4.0 release. According
> > to http://gcc.gnu.org/ml/gcc/2004-03/msg00463.html its unlikely :-(
> All you need is to create a pr with the wrong-code test case.
> The patch will be approved.
The patch was already committed when I wrote my mail. The patch solved
a PR which wasn't mentioned in the posted patch.
You suggested to add a REG_ALLOC_ORDER define. I now compared the
generated code from a cc1 with REG_ALLOC_ORDER defined and one without.
I think not defining REG_ALLOC_ORDER is better since with it defined
GCC puts constants between 1..127 into a even if it would be better
to use another data register. The with REG_ALLOC_ORDER prefered approach
might be better for small functions that only use scatch registers. IHMO,
for complex functions the old behaviour was better.