Re: Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-13 Thread Richard Biener
On Wed, 12 Nov 2014, Tom de Vries wrote: > On 12-11-14 15:17, Richard Biener wrote: > > On Wed, 12 Nov 2014, Tom de Vries wrote: > > > > > [ moved from gcc@ to gcc-patches@ ] > > > [ subject was: Re: [gomp4] openacc kernels directive support ] > > > On 30-09-14 15:37, Tom de Vries wrote: > > > >

Re: Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-12 Thread Tom de Vries
On 12-11-14 15:17, Richard Biener wrote: On Wed, 12 Nov 2014, Tom de Vries wrote: [ moved from gcc@ to gcc-patches@ ] [ subject was: Re: [gomp4] openacc kernels directive support ] On 30-09-14 15:37, Tom de Vries wrote: I would be happily accepting splitting the current autopar pass that way,

Re: Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-12 Thread Richard Biener
On Wed, 12 Nov 2014, Tom de Vries wrote: > [ moved from gcc@ to gcc-patches@ ] > [ subject was: Re: [gomp4] openacc kernels directive support ] > On 30-09-14 15:37, Tom de Vries wrote: > > > I would be happily accepting splitting the current autopar pass > > > that way, that is, do > > > > > > NE

Re: Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-12 Thread David Malcolm
On Wed, 2014-11-12 at 14:16 +0100, Tom de Vries wrote: I can't speak to the rest of the patch, but one readability nitpick: > diff --git a/gcc/passes.def b/gcc/passes.def > index 2305d67..dd91718 100644 > --- a/gcc/passes.def > +++ b/gcc/passes.def > @@ -241,6 +241,9 @@ along with GCC; see the fi

Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-12 Thread Tom de Vries
[ moved from gcc@ to gcc-patches@ ] [ subject was: Re: [gomp4] openacc kernels directive support ] On 30-09-14 15:37, Tom de Vries wrote: I would be happily accepting splitting the current autopar pass that way, that is, do NEXT_PASS (pass_parallelize_loops) PUSH_INSERT_PASSES_WITHIN (pass_paral