Re: [PATCH][C] Fix PR37985

2012-01-16 Thread Richard Guenther
On Mon, 16 Jan 2012, Richard Guenther wrote: > On Mon, 16 Jan 2012, Richard Guenther wrote: > > > On Fri, 13 Jan 2012, Joseph S. Myers wrote: > > > > > On Fri, 13 Jan 2012, Richard Guenther wrote: > > > > > > > This fixes PR37985 where since > > > > http://gcc.gnu.org/ml/gcc-patches/2006-08/ms

Re: [PATCH][C] Fix PR37985

2012-01-16 Thread Richard Guenther
On Mon, 16 Jan 2012, Richard Guenther wrote: > On Fri, 13 Jan 2012, Joseph S. Myers wrote: > > > On Fri, 13 Jan 2012, Richard Guenther wrote: > > > > > This fixes PR37985 where since > > > http://gcc.gnu.org/ml/gcc-patches/2006-08/msg01041.html we > > > mark conversions produced by convert_to_i

Re: [PATCH][C] Fix PR37985

2012-01-16 Thread Richard Guenther
On Fri, 13 Jan 2012, Joseph S. Myers wrote: > On Fri, 13 Jan 2012, Richard Guenther wrote: > > > This fixes PR37985 where since > > http://gcc.gnu.org/ml/gcc-patches/2006-08/msg01041.html we > > mark conversions produced by convert_to_integer with TREE_NO_WARNING. > > This may shadow "real" stmt

Re: [PATCH][C] Fix PR37985

2012-01-13 Thread Joseph S. Myers
On Fri, 13 Jan 2012, Richard Guenther wrote: > This fixes PR37985 where since > http://gcc.gnu.org/ml/gcc-patches/2006-08/msg01041.html we > mark conversions produced by convert_to_integer with TREE_NO_WARNING. > This may shadow "real" stmts with no effects, thus we should > simply strip them aga

[PATCH][C] Fix PR37985

2012-01-13 Thread Richard Guenther
This fixes PR37985 where since http://gcc.gnu.org/ml/gcc-patches/2006-08/msg01041.html we mark conversions produced by convert_to_integer with TREE_NO_WARNING. This may shadow "real" stmts with no effects, thus we should simply strip them again before checking for TREE_SIDE_EFFECTS. Bootstrap &