Re: Enable jump threading on paths meeting hot paths

2017-02-02 Thread Jeff Law
On 02/02/2017 06:49 AM, Jan Hubicka wrote: Hi, it seems I forgot to send the updated patch. Here it is. We now dump info like: Checking profitability of path: 5 (16 insns) 3 (2 insns) 34 (2 insns) 33 (4 insns) 32 (1 insns) 10 (3 insns) 6 Control statement insns: 16 Overall: 12 insns Regis

Re: Enable jump threading on paths meeting hot paths

2017-02-02 Thread Jan Hubicka
> > + if (!contains_hot_bb && speed_p && j < path_length - 1) > > j < path_length - 1 is already checked above? > > Otherwise looks ok. If it does fix the regression - does it? Thanks, yes it fixes the regression. Honza

Re: Enable jump threading on paths meeting hot paths

2017-02-02 Thread Richard Biener
On Thu, Feb 2, 2017 at 2:49 PM, Jan Hubicka wrote: > Hi, > it seems I forgot to send the updated patch. Here it is. > We now dump info like: > Checking profitability of path: 5 (16 insns) 3 (2 insns) 34 (2 insns) 33 (4 > insns) 32 (1 insns) 10 (3 insns) 6 > Control statement insns: 16 > Over

Re: Enable jump threading on paths meeting hot paths

2017-02-02 Thread Jan Hubicka
Hi, it seems I forgot to send the updated patch. Here it is. We now dump info like: Checking profitability of path: 5 (16 insns) 3 (2 insns) 34 (2 insns) 33 (4 insns) 32 (1 insns) 10 (3 insns) 6 Control statement insns: 16 Overall: 12 insns Registering FSM jump thread: (6, 10) incoming edge