OpenACC/OpenMP testing: Revise variable usage in constructs (was: Nested OpenACC/OpenMP constructs)

2014-12-17 Thread Thomas Schwinge
Hi! On Fri, 12 Dec 2014 21:04:21 +0100, I wrote: > On Wed, 10 Dec 2014 11:16:08 +0100, Jakub Jelinek wrote: > > On Wed, Dec 10, 2014 at 11:10:19AM +0100, Thomas Schwinge wrote: > > > --- /dev/null > > > +++ gcc/testsuite/c-c++-common/gomp/nesting-1.c > > > @@ -0,0 +1,77 @@ > > > +void > > > +f_om

Re: Nested OpenACC/OpenMP constructs

2014-12-12 Thread Thomas Schwinge
Hi! On Wed, 10 Dec 2014 11:16:08 +0100, Jakub Jelinek wrote: > On Wed, Dec 10, 2014 at 11:10:19AM +0100, Thomas Schwinge wrote: > > --- /dev/null > > +++ gcc/testsuite/c-c++-common/gomp/nesting-1.c > > @@ -0,0 +1,77 @@ > > +void > > +f_omp_parallel (void) > > +{ > > +#pragma omp parallel > > + {

Re: Nested OpenACC/OpenMP constructs (was: OpenACC GIMPLE_OACC_* -- or not?)

2014-12-10 Thread Jakub Jelinek
On Wed, Dec 10, 2014 at 11:10:19AM +0100, Thomas Schwinge wrote: > --- /dev/null > +++ gcc/testsuite/c-c++-common/gomp/nesting-1.c > @@ -0,0 +1,77 @@ > +void > +f_omp_parallel (void) > +{ > +#pragma omp parallel > + { > +int i; Can you please use a global variable declared outside of f_omp_pa

Re: Nested OpenACC/OpenMP constructs (was: OpenACC GIMPLE_OACC_* -- or not?)

2014-12-10 Thread Thomas Schwinge
Hi Jakub! On Wed, 10 Dec 2014 11:02:24 +0100, I wrote: > OpenACC: Rework nested constructs checking. > > gcc/ > * omp-low.c (scan_omp_target): Remove taskreg_nesting_level and > target_nesting_level assertions. > (check_omp_nesting_restrictions): Rework OpenACC con

Nested OpenACC/OpenMP constructs (was: OpenACC GIMPLE_OACC_* -- or not?)

2014-12-10 Thread Thomas Schwinge
ps at runtime, supporting #pragma omp directives inside of OpenACC > regions not needed (perhaps there are exceptions you want to support?). We have not yet tested such nested OpenACC/OpenMP constructs (one thing after the other), so I'm not confident to claim support for that, and earlier on