Re: [PATCH] Fix gimplify_expr ICE with Labels as Values (PR middle-end/79537)

2017-02-20 Thread Marek Polacek
On Mon, Feb 20, 2017 at 10:51:04AM +0100, Richard Biener wrote: > On Fri, Feb 17, 2017 at 2:53 PM, Marek Polacek wrote: > > We are crashing with a label as a value without accompanying goto. > > The problem is that TREE_SIDE_EFFECTS and FORCED_LABEL use the same > > ->base.side_effects_flag, so gi

Re: [PATCH] Fix gimplify_expr ICE with Labels as Values (PR middle-end/79537)

2017-02-20 Thread Richard Biener
On Fri, Feb 17, 2017 at 2:53 PM, Marek Polacek wrote: > We are crashing with a label as a value without accompanying goto. > The problem is that TREE_SIDE_EFFECTS and FORCED_LABEL use the same > ->base.side_effects_flag, so gimplify_expr is confused. We don't > ICE with 'goto *&&L;' becase then w

[PATCH] Fix gimplify_expr ICE with Labels as Values (PR middle-end/79537)

2017-02-17 Thread Marek Polacek
We are crashing with a label as a value without accompanying goto. The problem is that TREE_SIDE_EFFECTS and FORCED_LABEL use the same ->base.side_effects_flag, so gimplify_expr is confused. We don't ICE with 'goto *&&L;' becase then we take this path: 11406 case GOTO_EXPR: 11407