diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2025-01-25 10:28:23 +0100 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2025-01-25 10:29:50 +0100 |
| commit | 7eac20b0dde8df303557b6dd550d06ee98836c3c (patch) | |
| tree | 40f2b05cf5f0c4b29d1c8f114b9439b495ababcc /gcc/c/c-parser.cc | |
| parent | ce268ca2a923f8f35cc9dd5a7d0468a3980f129f (diff) | |
| download | gcc-7eac20b0dde8df303557b6dd550d06ee98836c3c.zip gcc-7eac20b0dde8df303557b6dd550d06ee98836c3c.tar.gz gcc-7eac20b0dde8df303557b6dd550d06ee98836c3c.tar.bz2 | |
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 <jakub@redhat.com>
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.
Diffstat (limited to 'gcc/c/c-parser.cc')
| -rw-r--r-- | gcc/c/c-parser.cc | 6 |
1 files changed, 1 insertions, 5 deletions
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<c_token> 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; |