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