From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugzilla@gcc.gnu.org>
Received: by sourceware.org (Postfix, from userid 48)
 id DFFCF388C025; Tue, 23 Feb 2021 15:18:09 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org DFFCF388C025
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/99208] [modules] ICE with partitions & instantiations of
 linkage-typedef structs
Date: Tue, 23 Feb 2021 15:18:09 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c++
X-Bugzilla-Version: 11.0
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: nathan at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-99208-4-W1CVOCwcFc@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-99208-4@http.gcc.gnu.org/bugzilla/>
References: <bug-99208-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-BeenThere: gcc-bugs@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-bugs mailing list <gcc-bugs.gcc.gnu.org>
List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-request@gcc.gnu.org?subject=help>
List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Tue, 23 Feb 2021 15:18:10 -0000

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Nathan Sidwell <nathan@gcc.gnu.org>:

https://gcc.gnu.org/g:47145e6916f58e1cd4f562c06fa7289da49f690c

commit r11-7347-g47145e6916f58e1cd4f562c06fa7289da49f690c
Author: Nathan Sidwell <nathan@acm.org>
Date:   Tue Feb 23 07:08:55 2021 -0800

    c++: typedef for linkage [PR 99208]

    Unnamed types with a typedef name for linkage were always troublesome
    in modules.  This is the underlying cause of that trouble -- we were
    creating incorrect type structures.  Classes have an implicit
    self-reference, and we created that for unnamed classes too.  It turns
    out we make use of this member, so just not generating it turned into
    a rathole.  This member is created using the anonymous name -- because
    we've not yet met the typedef name.  When we retrofit the typedef name
    we were checking identifier matching and changing all type variants
    with that identifier.  Which meant we ended up with a strange typedef
    for the self reference.  This fixes things to check for DECL identity
    of the variants, so we don't smash the self-reference -- that
    continues to have the anonymous name.

            PR c++/99208
            gcc/cp/
            * decl.c (name_unnamed_type): Check DECL identity, not IDENTIFI=
ER
            identity.
            gcc/testsuite/
            * g++.dg/modules/pr99208_a.C: New.
            * g++.dg/modules/pr99208_b.C: New.=