This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [committed] Clean up Fortran OpenACC wait clause handling
- From: Thomas Schwinge <thomas at codesourcery dot com>
- To: Julian Brown <julian at codesourcery dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>, <fortran at gcc dot gnu dot org>, Chung-Lin Tang <cltang at codesourcery dot com>, Jakub Jelinek <jakub at redhat dot com>
- Date: Thu, 13 Dec 2018 17:03:05 +0100
- Subject: Re: [committed] Clean up Fortran OpenACC wait clause handling
- References: <874lbythpn.fsf@euler.schwinge.homeip.net> <20181203160223.4500f213@squid.athome>
Hi Julian!
On Mon, 3 Dec 2018 16:02:23 +0000, Julian Brown <julian@codesourcery.com> wrote:
> On Fri, 30 Nov 2018 21:48:20 +0100
> Thomas Schwinge <thomas@codesourcery.com> wrote:
> > commit 3e3de40a5ab21d72f08071a7a40120dd05608cc1
> > Author: tschwinge <tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4>
> > Date: Fri Nov 30 20:39:18 2018 +0000
> >
> > Clean up Fortran OpenACC wait clause handling
> >
> > "wait" can be deduced from "wait_list".
> >
> > gcc/fortran/
> > * gfortran.h (struct gfc_omp_clauses): Remove "wait".
> > Adjust all users.
>
> This appears to conflict with Chung-Lin's uncommitted patch ("Properly
> handle wait clause with no arguments"):
>
> https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01973.html
>
> I'm not sure if such waits have a "wait_list" or not -- I guess not
> though?
Right; that's what I provided an incremental patch for in my review in
<https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00481.html>, and asked
Chung-Lin (and he agreed) to incorporate.
Grüße
Thomas