[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