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

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Jakub Jelinek
<ja...@gcc.gnu.org>:

https://gcc.gnu.org/g:5de999df9fa0134a1621b552eb2abd65a6384ffd

commit r12-9138-g5de999df9fa0134a1621b552eb2abd65a6384ffd
Author: Jakub Jelinek <ja...@redhat.com>
Date:   Thu Jan 5 11:57:30 2023 +0100

    openmp: Fix up finish_omp_target_clauses [PR108286]

    The comment in the loop says that we shouldn't add a map clause if such
    a clause exists already, but the loop was actually using OMP_CLAUSE_DECL
    on any clause.  Target construct can have various clauses which don't
    have OMP_CLAUSE_DECL at all (e.g. nowait, device or if) or clause
    where it means something different (e.g. privatization clauses, allocate,
    depend).

    So, only check OMP_CLAUSE_DECL on OMP_CLAUSE_MAP clauses.

    2023-01-05  Jakub Jelinek  <ja...@redhat.com>

            PR c++/108286
            * semantics.cc (finish_omp_target_clauses): Ignore clauses other
than
            OMP_CLAUSE_MAP.

            * testsuite/libgomp.c++/pr108286.C: New test.

    (cherry picked from commit 29c3218618ef6177dc33871b26c8fbd9b21eabe1)
  • [Bug c++/108286] [12 Regression... cvs-commit at gcc dot gnu.org via Gcc-bugs

Reply via email to