[patch] Fix for PR26222

Andrew Pinski pinskia@physics.uc.edu
Sun Feb 12 17:36:00 GMT 2006


On Feb 12, 2006, at 4:20 AM, Zdenek Dvorak wrote:

> Hello,
>
> the part of the code in loop (insn 94) is replaced with
>
> (set (mem/c/i:SI (plus:SI (reg/f:SI 113 sfp)
>                           (const_int 64 [0x40])) [9 S4 A32])
>      (reg:SI 194))
>
> Which indeed does not help too much.  Nevertheless, there is no easy 
> way
> how to detect this case, and I do not intend to add any pass that would
> further optimize this.  Invariant motion for this construction should
> happen on tree level (which seems to happen for the C testcase, but
> probably because of exceptions not for the Java one).

After trying the patch, there is just an extra store and no extra load 
so
it is not as bad as I originally thought it would be.  I am not that
worried about the extra store as I was about the extra load.
Though I wonder why RTL level DSE did not remove it (unless it was
not really unused until after rtl level DSE was run).

Thanks,
Andrew Pinski



More information about the Gcc-patches mailing list