[Bug c/34506] New: gcc disallows comma between C/C++ OpenMP clauses

[Bug c/34506] New: gcc disallows comma between C/C++ OpenMP clauses

[jakub at gcc dot gnu dot org]

OpenMP 2.5 in 2.1 has:
#pragma omp directive-name [clause[ [,] clause]...] new-line
and for Fortran
sentinel directive-name [clause[[,] clause]...]
While for Fortran the optional commas are accepted, the C/C++ parser rejects
them.

Working on a patch.


-- 
           Summary: gcc disallows comma between C/C++ OpenMP clauses
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Keywords: openmp
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: jakub at gcc dot gnu dot org
        ReportedBy: jakub at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34506

[Bug c/34506] gcc disallows comma between C/C++ OpenMP clauses

[jakub at gcc dot gnu dot org]

------- Comment #1 from jakub at gcc dot gnu dot org  2007-12-17 14:17 -------
Subject: Bug 34506

Author: jakub
Date: Mon Dec 17 14:16:42 2007
New Revision: 131008

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131008
Log:
        PR c/34506
        * c-parser.c (c_parser_omp_all_clauses): Accept optional comma
        in between clauses.

        * parser.c (cp_parser_omp_all_clauses): Accept optional comma
        in between clauses.

        * gcc.dg/gomp/clause-2.c: New test.
        * g++.dg/gomp/clause-4.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/gomp/clause-4.C
    trunk/gcc/testsuite/gcc.dg/gomp/clause-2.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-parser.c
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/parser.c
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34506

[Bug c/34506] gcc disallows comma between C/C++ OpenMP clauses

[jakub at gcc dot gnu dot org]

------- Comment #2 from jakub at gcc dot gnu dot org  2007-12-17 14:40 -------
Fixed on the trunk so far.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-12-17 14:40:16
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34506

[Bug c/34506] gcc disallows comma between C/C++ OpenMP clauses

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2007-12-23 19:12 -------
Fixed so closing.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
           Keywords|                            |rejects-valid
         Resolution|                            |FIXED
   Target Milestone|---                         |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34506