On Wed, Nov 04, 2015 at 06:20:06PM +0100, Thomas Schwinge wrote:
> Hi Jakub!
> 
> On Wed, 4 Nov 2015 11:30:28 +0100, Jakub Jelinek <ja...@redhat.com> wrote:
> > >  gfc_match_oacc_update (void)
> > >  {
> > >    gfc_omp_clauses *c;
> > > +  locus here = gfc_current_locus;
> > > +
> > >    if (gfc_match_omp_clauses (&c, OACC_UPDATE_CLAUSES, false, false, true)
> > >        != MATCH_YES)
> > >      return MATCH_ERROR;
> > >  
> > > +  if (!c->lists[OMP_LIST_MAP])
> > > +    {
> > > +      gfc_error ("%<acc update%> must contain at least one "
> > > +          "%<device%> or %<host/self%> clause at %L", &here);
> > 
> > There is no host/self clause I'd guess, so you should spell those
> > separately.
> 
> That's the same language as used in the C and C++ front ends; the host
> and self clauses are synonymous.

Then it should be %<host%>/%<self%> at least, or better
%<host%> or %<self%>.
Both in Fortran and in the C/C++ FEs.

        Jakub

Reply via email to