On Wed, May 12, 2021 at 04:36:00PM +0200, Marcel Vollweiler wrote:
> +       c_parser_error (parser, "%<#pragma omp target%> with map-type-"
> +                               "modifier other than %<always%> or %<close%>"

Please just write modifier rather than map-type-modifier

> +       cp_parser_error (parser, "%<#pragma omp target%> with map-type-"

Likewise.

> +                                "modifier other than %<always%> or %<close%>"
> +                                "on %<map%> clause");
> +       cp_parser_skip_to_closing_parenthesis (parser,
> +                                              /*recovering=*/true,
> +                                              /*or_comma=*/false,
> +                                              /*consume_paren=*/true);
> +           return list;

And this line looks misindented, previous is tab + 2 spaces, this one
tab + 6 spaces, should be tab + 4 spaces.

Ok for trunk with those changes (and adjusting the testcases accordingly).

        Jakub

Reply via email to