Re: [PING][PATCH][REVISED] Fix PR middle-end/PR28690, modify swap_commutative_operands_p

["H. J. Lu" <hjl at lucon dot org>]

On Thu, Jun 21, 2007 at 06:14:12PM +0200, Michael Matz wrote:
> Hi,
> 
> On Thu, 21 Jun 2007, H. J. Lu wrote:
> 
> > > I had to refresh the patch (attached below) since the last time I 
> > > posted it due to the dataflow merge.  I bootstrapped and regtested it 
> > > on powerpc64-linux and everything still looks good.  Is this patch ok 
> > > for mainline once the freeze is over?
> > 
> > Gcc 4.3 revision 125901 miscompiles 447.dealII in SPEC CPU 2006 and 
> > revision 125849 is OK. I have to rebuild my gcc with revision 125849 and 
> > restart my SPEC 2000/2006 runs. It will take a while.
> 
> Note that trunk miscomples dealII anyway, see PR32303 (an instance of 
> PR30252), which is fixed (or worked around depending on whom you ask) in 
> 4.2 but not on trunk.  The nature of the bug makes it possible that it's 
> randomly exposed or hidden by unrelated changes.

-O2 has been OK on dealII.


H.J.