http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51466

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-12-08 
15:41:50 UTC ---
(In reply to comment #2)
> Created attachment 26024 [details]
> gcc47-pr51466.patch
> 
> Another alternative.  Note that apparently gimple_rhs_has_side_effects has 
> been
> removed in the mean time, still having TREE_SIDE_EFFECTS consistent doesn't
> hurt.

Hm, well.  It doesn't hurt, yes.  But maybe it confuses people as to what
is actually needed to be kept up-to-date?  (that's not 100% clear to me
either - GIMPLE at least does not care about TREE_SIDE_EFFECTS, though
older RTL expansion code might).

Reply via email to