Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-10 Thread Andrew Pinski
On Fri, Jul 8, 2016 at 12:16 AM, Richard Biener wrote: > > This is a final candidate patch to add code-hoisting to GIMPLE. > > I've already committed several patches fixing fallout and the following > one adds -fno-code-hoisting (I renamed the option) to a few testcases. > I filed PRs for the case

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Jeff Law
On 07/08/2016 01:30 AM, Richard Biener wrote: On Mon, 4 Jul 2016, Steven Bosscher wrote: On Mon, Jul 4, 2016 at 1:26 PM, Richard Biener wrote: The following patch is Stevens code-hoisting based on PRE forward-ported and fixed for bootstrap plus the case of hoisting code across loops which we

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Rainer Orth
Hi Richard, >> I've just bootstrapped the patch on sparc-sun-solaris2.12, which >> uncovered a couple of testsuite failures: >> >> +FAIL: gcc.dg/tree-ssa/split-path-5.c scan-tree-dump-times split-paths >> "Duplicat >> ing join block" 2 >> +FAIL: gcc.dg/tree-ssa/split-path-5.c scan-tree-dump-times

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Richard Biener
On Fri, 8 Jul 2016, Rainer Orth wrote: > Hi Richard, > > > This is a final candidate patch to add code-hoisting to GIMPLE. > > > > I've already committed several patches fixing fallout and the following > > one adds -fno-code-hoisting (I renamed the option) to a few testcases. > > I filed PRs for

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Rainer Orth
Hi Richard, > This is a final candidate patch to add code-hoisting to GIMPLE. > > I've already committed several patches fixing fallout and the following > one adds -fno-code-hoisting (I renamed the option) to a few testcases. > I filed PRs for the cases code-hoisting exposes missed optimization >

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Richard Biener
On Mon, 4 Jul 2016, Steven Bosscher wrote: > On Mon, Jul 4, 2016 at 1:26 PM, Richard Biener wrote: > > > > The following patch is Stevens code-hoisting based on PRE forward-ported > > and fixed for bootstrap plus the case of hoisting code across loops > > which we generally do not want (expression

[PATCH] Add code-hoisting to GIMPLE

2016-07-08 Thread Richard Biener
This is a final candidate patch to add code-hoisting to GIMPLE. I've already committed several patches fixing fallout and the following one adds -fno-code-hoisting (I renamed the option) to a few testcases. I filed PRs for the cases code-hoisting exposes missed optimization opportunities in passe

Re: [PATCH] Add code-hoisting to GIMPLE

2016-07-04 Thread Steven Bosscher
On Mon, Jul 4, 2016 at 1:26 PM, Richard Biener wrote: > > The following patch is Stevens code-hoisting based on PRE forward-ported > and fixed for bootstrap plus the case of hoisting code across loops > which we generally do not want (expressions in the loop exit target block > are antic-in through

[PATCH] Add code-hoisting to GIMPLE

2016-07-04 Thread Richard Biener
The following patch is Stevens code-hoisting based on PRE forward-ported and fixed for bootstrap plus the case of hoisting code across loops which we generally do not want (expressions in the loop exit target block are antic-in throughout the whole loop unless they are killed and thus get inserted