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

--- Comment #4 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tobias Burnus <[email protected]>:

https://gcc.gnu.org/g:26e33a3c47ab54612f77dbfbd9827d729e8cfa41

commit r17-5029-g26e33a3c47ab54612f77dbfbd9827d729e8cfa41
Author: Tobias Burnus <[email protected]>
Date:   Fri Oct 9 17:57:21 2026 +0200

    OpenMP: Move 'sorry, unimplemented' for simdlen's 'scaled' modifier
[PR127723]

    This is a fixup to commit r17-4947-gfbcafeec06a661.

    It turned out that the sorry message did not show up on system without
    targetm.simd_clone.compute_vecsize_and_simdlen; hence, move the check to
the
    FE - for C/C++ to c_omp_declare_simd_clauses_to_numbers - such that it
always
    triggers. (Fortran, 'scaled' modifier support still has to be added.)

    Note: It would be also fine not to sorry on those, but instead of finding
    an effective target for this, moving the sorry was simpler. Note
additionally
    that 'simd' is effectively just a hint - such that silently ignoring
'scaled'
    would also work, but the current solution seems to make more sense until
there
    is some actual support implemented on the vectorizer side.

    The commit also fixes a not-fully working last-minute line-number change in
    simdlen-scaled-4.c, which caused it to fail also when the hook was present.

            PR middle-end/127723

    gcc/c-family/ChangeLog:

            * c-omp.cc (c_omp_declare_simd_clauses_to_numbers): Add sorry
            diagnostic for simdlen with scaled modifer, moved from
            omp-simd-clone.cc to ensure it is always active.

    gcc/ChangeLog:

            * omp-simd-clone.cc (simd_clone_clauses_extract): Remove sorry
            diagnostic for simdlen with scaled modifier, moved to
            c-family/c-omp.cc to ensure it is always active not only when
            a target hook exists.

    libgomp/ChangeLog:

            * libgomp.texi (OpenMP 6.1 Impl. Status): Link to Bugzilla
            for the pending 'scaled' modifer to the 'simdlen' clause.

    gcc/testsuite/ChangeLog:

            * c-c++-common/gomp/simdlen-scaled-1.c: Update dg-message sorry
            handling.
            * c-c++-common/gomp/simdlen-scaled-4.c: Ditto.
            * g++.dg/gomp/simdlen-scaled-1.C: Ditto.
            * g++.dg/gomp/simdlen-scaled-2.C: Ditto.

Reply via email to