Re: [PATCH] Fix PR middle-end/81194, ICE during RTL pass: expand

2017-06-29 Thread Peter Bergner
On 6/29/17 3:51 AM, Richard Biener wrote: > On Thu, Jun 29, 2017 at 10:34 AM, Richard Biener > To answer myself the unreachable case vanishes at > execute_cleanup_cfg_post_optimizing > via group_case_labels. find_taken_edge wouldn't handle this case either. > > I am testing a patch fixing both -

Re: [PATCH] Fix PR middle-end/81194, ICE during RTL pass: expand

2017-06-29 Thread Richard Biener
On Thu, Jun 29, 2017 at 10:34 AM, Richard Biener wrote: > On Thu, Jun 29, 2017 at 5:01 AM, Peter Bergner wrote: >> With the fix to PR51513 and follow on fixes for PR80707, PR80775 and PR80823, >> we can now end up with switch statements that contain nothing but a default >> case statement. The e

Re: [PATCH] Fix PR middle-end/81194, ICE during RTL pass: expand

2017-06-29 Thread Richard Biener
On Thu, Jun 29, 2017 at 5:01 AM, Peter Bergner wrote: > With the fix to PR51513 and follow on fixes for PR80707, PR80775 and PR80823, > we can now end up with switch statements that contain nothing but a default > case statement. The expand_case() function contains code that assumes we > have at

[PATCH] Fix PR middle-end/81194, ICE during RTL pass: expand

2017-06-28 Thread Peter Bergner
With the fix to PR51513 and follow on fixes for PR80707, PR80775 and PR80823, we can now end up with switch statements that contain nothing but a default case statement. The expand_case() function contains code that assumes we have at least one non-default case statement, leading to the ICE report