[Bug c/87911] OpenACC/OpenMP clauses parsing: comma operator vs. separator

[Bug c/87911] OpenACC/OpenMP clauses parsing: comma operator vs. separator

[tschwinge at gcc dot gnu.org]

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

Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |burnus at gcc dot gnu.org
   Last reconfirmed|                            |2021-07-01
           See Also|                            |https://github.com/OpenACC/
                   |                            |openacc-spec/pull/363
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW

--- Comment #2 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
For OpenACC this got discussed/resolved in
<https://github.com/OpenACC/openacc-spec/pull/363> "Issue 354: Disallow C/C++
comma operators" (similar to the OpenMP wording), and -- if I remember
correctly -- Tobias verified that the GCC/OpenACC implementation behaves
properly?

Don't know about the OpenMP status.

[Bug c/87911] OpenACC/OpenMP clauses parsing: comma operator vs. separator

[jakub at gcc dot gnu.org]

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
For OpenMP, I think after the clarification it got fixed in
r9-3992-g81a227c6054a82118d80d3f4b2fffb44bc43aae2
There is still the fuzzy case whether #pragma omp declare reduction combiner
expression should allow commas or not, all compilers I've checked do allow it
there and IMNSHO it is the right thing, but maybe the standard doesn't allow
that, but it is the standard that should be fixed in that case.
#pragma omp declare reduction(+:struct S:omp_out.x += omp_in.x, omp_out.y +=
omp_in.y)