From 7eac20b0dde8df303557b6dd550d06ee98836c3c Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Sat, 25 Jan 2025 10:28:23 +0100 Subject: c: Diagnose ,) at the end of OpenMP clauses [PR118639] This is something the C++ FE has been diagnosing but C FE only complained if there wasn't an identifier right after opening ( 2025-01-25 Jakub Jelinek PR c/118639 * c-parser.cc (c_parser_omp_variable_list): Remove first variable and emit "expected identifier" error regardless of it. * c-c++-common/gomp/pr118639.c: New test. * c-c++-common/goacc/cache-2.c: Remove one xfail for c. --- gcc/c/c-parser.cc | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'gcc/c') diff --git a/gcc/c/c-parser.cc b/gcc/c/c-parser.cc index 93da0fb..4f40dfc 100644 --- a/gcc/c/c-parser.cc +++ b/gcc/c/c-parser.cc @@ -16307,7 +16307,6 @@ c_parser_omp_variable_list (c_parser *parser, auto_vec tokens; unsigned int tokens_avail = 0; c_token *saved_tokens = NULL; - bool first = true; while (1) { @@ -16343,7 +16342,6 @@ c_parser_omp_variable_list (c_parser *parser, break; c_parser_consume_token (parser); - first = false; continue; } @@ -16507,8 +16505,7 @@ c_parser_omp_variable_list (c_parser *parser, t = c_parser_predefined_identifier (parser).value; else { - if (first) - c_parser_error (parser, "expected identifier"); + c_parser_error (parser, "expected identifier"); break; } @@ -16702,7 +16699,6 @@ c_parser_omp_variable_list (c_parser *parser, break; c_parser_consume_token (parser); - first = false; } return list; -- cgit v1.1