On Fri, Mar 16, 2018 at 12:53:29PM +0100, Richard Biener wrote: > > An alternative slightly more expensive patch is the following which > > I'm now testing on x86_64-unknown-linux-gnu, the above testcase > > is fixed with it (verified with a cross). > > Bootstrapped and tested on x86_64-unknown-linux-gnu, ok? > > > 2018-03-16 Richard Biener <rguent...@suse.de> > > > > PR c/84873 > > * c-gimplify.c (c_gimplify_expr): Revert previous change. Instead > > unshare the possibly folded expression. > > > > + /* Make sure to unshare the result, tree sharing is invalid > > + during gimplification. */ > > + *op1_p = unshare_expr (convert (unsigned_type_node, *op1_p));
Ok, thanks. Jakub