This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR middle-end/71373] Handle more OMP_CLAUSE_* in nested function decomposition
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org, fortran at gcc dot gnu dot org
- Date: Thu, 2 Jun 2016 18:25:17 +0200
- Subject: Re: [PR middle-end/71373] Handle more OMP_CLAUSE_* in nested function decomposition
- Authentication-results: sourceware.org; auth=none
- References: <87h9ddrlrx dot fsf at hertz dot schwinge dot homeip dot net> <5459732B dot 1010101 at codesourcery dot com> <20160601151217 dot GS28550 at tucnak dot redhat dot com> <87d1nzsgt2 dot fsf at hertz dot schwinge dot homeip dot net>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jun 02, 2016 at 06:20:57PM +0200, Thomas Schwinge wrote:
> relevant. Nested function decomposition is not applicable to C++, so we
> don't need any C++ test cases, right?
C++ has lambdas, but those are already lowered in the FE, so yes,
from the OpenMP/OpenACC FEs, tree-nested.c is only used by C and Fortran.
> [PR middle-end/71373] Handle more OMP_CLAUSE_* in nested function decomposition
>
> gcc/
> * gimplify.c (gimplify_adjust_omp_clauses): Discard
> OMP_CLAUSE_TILE.
> * omp-low.c (scan_sharing_clauses): Don't expect OMP_CLAUSE_TILE.
> gcc/testsuite/
> * c-c++-common/goacc/combined-directives.c: XFAIL tree scanning
> for OpenACC tile clauses.
> * gfortran.dg/goacc/combined-directives.f90: Likewise.
>
> gcc/
> PR middle-end/71373
> * tree-nested.c (convert_nonlocal_omp_clauses)
> (convert_local_omp_clauses): Handle OMP_CLAUSE_ASYNC,
> OMP_CLAUSE_WAIT, OMP_CLAUSE_INDEPENDENT, OMP_CLAUSE_AUTO,
> OMP_CLAUSE__CACHE_, OMP_CLAUSE_TILE.
> gcc/testsuite/
> PR middle-end/71373
> * gcc.dg/goacc/nested.c: New file.
> * gcc.dg/goacc/pr71373.c: Likewise.
> * gfortran.dg/goacc/subroutines.f90: Update.
LGTM.
Jakub