[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-09-08
           Assignee|unassigned at gcc dot gnu.org      |jakub at gcc dot gnu.org
   Target Milestone|---                         |5.3
     Ever confirmed|0                           |1

[Bug c/67501] New: Bad error recovery for invalid OpenMP clauses in C FE

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501

            Bug ID: 67501
           Summary: Bad error recovery for invalid OpenMP clauses in C FE
           Product: gcc
           Version: 5.2.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: jakub at gcc dot gnu.org
  Target Milestone: ---

/* */
/* { dg-do compile } */
/* { dg-options "-fopenmp" } */

void
foo (void)
{
  int i;
  #pragma omp for simd simdlen(4
  for (i = 0; i < 16; ++i)
    ;
}

ICEs during clause splitting, the C++ FE handles this right.

[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed Sep  9 07:24:48 2015
New Revision: 227578

URL: https://gcc.gnu.org/viewcvs?rev=227578&root=gcc&view=rev
Log:
        PR c/67501
        * c-parser.c (c_parser_oacc_all_clauses,
        c_parser_omp_all_clauses): Remove invalid clause from
        list of clauses even if parser->error is set.

        * c-c++-common/gomp/pr67501.c: New test.

Added:
    trunk/gcc/testsuite/c-c++-common/gomp/pr67501.c
Modified:
    trunk/gcc/c/ChangeLog
    trunk/gcc/c/c-parser.c
    trunk/gcc/testsuite/ChangeLog

[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed Sep  9 07:31:43 2015
New Revision: 227582

URL: https://gcc.gnu.org/viewcvs?rev=227582&root=gcc&view=rev
Log:
        PR c/67501
        * c-parser.c (c_parser_oacc_all_clauses,
        c_parser_omp_all_clauses): Remove invalid clause from
        list of clauses even if parser->error is set.

        * c-c++-common/gomp/pr67501.c: New test.

Added:
    branches/gcc-5-branch/gcc/testsuite/c-c++-common/gomp/pr67501.c
Modified:
    branches/gcc-5-branch/gcc/c/ChangeLog
    branches/gcc-5-branch/gcc/c/c-parser.c
    branches/gcc-5-branch/gcc/testsuite/ChangeLog

[Bug c/67501] Bad error recovery for invalid OpenMP clauses in C FE

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67501

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed for 5.3+.