[Bug middle-end/103642] [12 Regression] ICE in lower_omp_target: omp-low.c:12977 (fold_convert_loc) for omp target map(from: t.s->a[:N]) since r12-5835-g0ab29cf0bb68960c

["cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>]

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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Chung-Lin Tang <cltang@gcc.gnu.org>:

https://gcc.gnu.org/g:1c91b014923f418e0aab789c5cf57facf04bf266

commit r12-6890-g1c91b014923f418e0aab789c5cf57facf04bf266
Author: Chung-Lin Tang <cltang@codesourcery.com>
Date:   Thu Jan 27 18:33:00 2022 +0800

    Fix omp-low ICE for indirect references based off component access
[PR103642]

    This issue was triggered after the patch extending syntax for component
access
    in map clauses in commit 0ab29cf0bb68960c1f87405f14b4fb2109254e2f.

    In gimplify_scan_omp_clauses, the case for handling indirect accesses
(which
    creates firstprivate ptr and zero-length array section map for such decls)
was
    erroneously went into for non-pointer cases (here being the base struct
decl),
    so added the
    appropriate checks there.

    Added new testcase is a compile only test for the ICE. The original
omptests
    t-partial-struct test actually should not execute correctly, because for
    map(t.s->a[:N]), map(t.s[:1]) is not implicitly mapped, thus the entire
    offloaded access does not work as is (fixing that omptests test is out of
    scope here).

    2022-01-27  Chung-Lin Tang  <cltang@codesourcery.com>

            PR middle-end/103642

    gcc/ChangeLog:

            * gimplify.cc (gimplify_scan_omp_clauses): Do not do indir_p
handling
            for non-pointer or non-reference-to-pointer cases.

    gcc/testsuite/ChangeLog:

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