Re: [PATCH] Fix a FSM threading ICE (PR tree-optimization/65735)

2015-04-11 Thread Jakub Jelinek
On Sat, Apr 11, 2015 at 07:19:00PM +0200, Marc Glisse wrote: > On Sat, 11 Apr 2015, Jakub Jelinek wrote: > > >@@ -1391,13 +1391,13 @@ thread_through_normal_block (edge e, > > vec *bb_path; > > vec_alloc (bb_path, n_basic_blocks_for_fn (cfun)); > > vec_safe_push (bb_path, e->dest); >

Re: [PATCH] Fix a FSM threading ICE (PR tree-optimization/65735)

2015-04-11 Thread Richard Biener
On April 11, 2015 6:34:43 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >On the following testcase, starting with r221675 aka PR65177 fix >we get ICE, because FSM discovery finds a path that includes the same >blocks >multiple times, like: >Registering FSM jump thread: (9, 4) incoming edge; (4, 5) (

Re: [PATCH] Fix a FSM threading ICE (PR tree-optimization/65735)

2015-04-11 Thread Marc Glisse
On Sat, 11 Apr 2015, Jakub Jelinek wrote: @@ -1391,13 +1391,13 @@ thread_through_normal_block (edge e, vec *bb_path; vec_alloc (bb_path, n_basic_blocks_for_fn (cfun)); vec_safe_push (bb_path, e->dest); - hash_set *visited_phis = new hash_set; + hash_set *visited_bbs =

[PATCH] Fix a FSM threading ICE (PR tree-optimization/65735)

2015-04-11 Thread Jakub Jelinek
Hi! On the following testcase, starting with r221675 aka PR65177 fix we get ICE, because FSM discovery finds a path that includes the same blocks multiple times, like: Registering FSM jump thread: (9, 4) incoming edge; (4, 5) (5, 12) (12, 14) (14, 5) (5, 12) nocopy; (5, 12) All these bbs b