http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47614
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #23288|0 |1
is obsolete| |
--- Comment #13 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-09
18:48:46 UTC ---
Created attachment 23289
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23289
gcc46-pr47614-2.patch
Actually, the second patch is not enough, we don't delete the insn in that
case, but still reload_cse_simplify_operands optimizes the MEM into a register,
ignoring its side-effects.
I think this should work. Alternatively, if we go the first patch route, I
think side_effects_p guard in reload_cse_simplify_operands is still desirable.
