This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Combine performance regression (was: Fix PR target/18701)


On Tue, 25 Jan 2005, Roger Sayle wrote:
> Completely removing the above clause from combine.c on mainline
> bootstraps all default languages and regression tests with no new
> failures on both i686-pc-linux-gnu and powerpc-unknown-linux-gnu.
>
> I think this code, that was added by Jakub Jelinek to resolve
> PR optimization/6086, was really papering over a bug elsewhere
> in the compiler, more specifically in reload.  Given that the
> testcase that was added for that PR, g++.dg/opt/preinc1.C, now
> still passes on PPC even with the above clause removed,

Hm, does that test also pass with the other patch (the right one ;-)
for PR target/18701 reverted (cvs diff -r1.46{3,4} combine.c)?

brgds, H-P


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]