Re: [PATCH] Allow FSM to thread single block cases too

2015-10-15 Thread Richard Biener
On Wed, Oct 14, 2015 at 5:53 PM, Jeff Law wrote: > On 10/14/2015 09:43 AM, Jan Hubicka wrote: >> >> I think he asked for trivial forward threads though due to repeated >> tests. >> I hacked FRE to do this (I think), but maybe some trivial cleanup >> opportunities >> are sti

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Jeff Law
On 10/14/2015 09:43 AM, Jan Hubicka wrote: I think he asked for trivial forward threads though due to repeated tests. I hacked FRE to do this (I think), but maybe some trivial cleanup opportunities are still left here. Honza? Well, unthreaded jumps quite confuse profile prediction and create p

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Jan Hubicka
> >>> I think he asked for trivial forward threads though due to repeated > >>> tests. > >>> I hacked FRE to do this (I think), but maybe some trivial cleanup > >>> opportunities > >>> are still left here. Honza? Well, unthreaded jumps quite confuse profile prediction and create profiles that we

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Jeff Law
On 10/14/2015 06:46 AM, Richard Biener wrote: This or other related patches in the range r228731:228774 has caused a quite big jump in SPEC CPU 2000 binary sizes (notably 176.gcc - so maybe affecting bootstrap as well, at -O3). Are you sure this doesn't re-introduce DOM effectively peeling all l

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Richard Biener
On Wed, Oct 14, 2015 at 2:42 PM, Jeff Law wrote: > On 10/14/2015 04:16 AM, Richard Biener wrote: >> >> On Tue, Oct 13, 2015 at 2:52 PM, Richard Biener >> wrote: >>> >>> On Tue, Oct 13, 2015 at 2:21 PM, Jeff Law wrote: One of the cases that was missing in the FSM support is threadi

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Jeff Law
On 10/14/2015 04:16 AM, Richard Biener wrote: On Tue, Oct 13, 2015 at 2:52 PM, Richard Biener wrote: On Tue, Oct 13, 2015 at 2:21 PM, Jeff Law wrote: One of the cases that was missing in the FSM support is threading when the path is a single block. ie, a control statement's output can be st

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-14 Thread Richard Biener
fectively peeling all loops once? Richard. > Richard. > >> Bootstrapped and regression tested on x86_64-linux-gnu. Installed on the >> trunk. >> >> Jeff >> >> commit a53bb29a1dffd329aa6235b88b0c2a830aa5a59e >> Author: Jeff Law

Re: [PATCH] Allow FSM to thread single block cases too

2015-10-13 Thread Richard Biener
4-linux-gnu. Installed on the > trunk. > > Jeff > > commit a53bb29a1dffd329aa6235b88b0c2a830aa5a59e > Author: Jeff Law > Date: Tue Oct 13 06:19:20 2015 -0600 > > [PATCH] Allow FSM to thread single block cases too > > * tree-ssa-threadbackward.c > (f

[PATCH] Allow FSM to thread single block cases too

2015-10-13 Thread Jeff Law
ested on x86_64-linux-gnu. Installed on the trunk. Jeff commit a53bb29a1dffd329aa6235b88b0c2a830aa5a59e Author: Jeff Law Date: Tue Oct 13 06:19:20 2015 -0600 [PATCH] Allow FSM to thread single block cases too * tree-ssa-threadbackward.c (fsm_find_control_statement_thread_paths)