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