On Wed, Apr 28, 2021 at 10:26:44PM +0200, Tobias Burnus wrote:
> On 28.04.21 15:41, Jakub Jelinek wrote:
> > > @@ -261,6 +263,7 @@ gfc_match_omp_variable_list (const char *str, 
> > > gfc_omp_namelist **list,
> > > +      gfc_gobble_whitespace ();
> > >        if ((allow_sections && gfc_peek_ascii_char () == '(')
> > >            || (allow_derived && gfc_peek_ascii_char () == '%')
> > Is this change specific to depend/affinity or iterators?
> > If not, shouldn't it go in separately and with a testcase that shows when it
> > is needed?
> 
> I encountered it when writing a testcase - and the testcase was using it.
> However, if you think it makes sense to have it separately.
> 
> It fixes the issue:
> 
>     7 | !$omp target enter data map( to: a (:) )
>       |                                   1
> Error: Syntax error in OpenMP variable list at (1)
> 
> OK for mainline?

Yes, thanks.
And IMHO also should be backported at least to 11.

        Jakub

Reply via email to