Reload bug & SRA oddness

Roman Zippel zippel@linux-m68k.org
Tue May 1 16:24:00 GMT 2007


Hi,

On Tue, 1 May 2007, Alexandre Oliva wrote:

> On Apr 30, 2007, Bernd Schmidt <bernds_cb1@t-online.de> wrote:
> 
> > As a consequence of all these problems, I'd like to see this patch
> > reverted for now until it we know how to do it properly.
> 
> Here's a new patch that I hope will correct all the regressions,
> including those on platforms I don't have access to.  Bootstrapped and
> regtested on x86_64-linux-gnu.  Ok to install?

This still breaks this m68k test case:
http://gcc.gnu.org/ml/gcc/2007-04/msg00830.html

BTW the previous patch broke the m68k bootstrap and this doesn't make me 
very confident this one will do better.

A specific comment about the patch, it might be better to keep combining 
multiple fields, if they don't match the alignment exactly (e.g. as in the 
above test case).
Even better would be if the per field copy of the sra pass would be 
completely suppressed if the structure is already likely to be in a 
register anyway.

bye, Roman



More information about the Gcc-patches mailing list