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 8C4C43858C74; Thu,  9 Mar 2023 13:51:37 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8C4C43858C74
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1678369897;
	bh=1n5fYHqunhbF8IZ7ISj1qGZaDxsEFTRTKj+Pp1MrTmM=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=or6iao+fjT8gxFSfnaOFrj7jCrIz8MoUJZ5A/jnRz9IJ7ZdOU5xb/0IeRWHmg6c5h
	 h7YJ/cmISPt9LgOp00kSFTVPM9E5WMD622VSy5odnZqED9mRar2XD8N8DBBKe2VP03
	 S5TnhOIyKEfEZghdpj+HaLA3nNO507fmjVpth6OY=
From: "hubicka at ucw dot cz" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/108887] [13 Regression] ICE in
 process_function_and_variable_attributes since r13-3601
Date: Thu, 09 Mar 2023 13:51:36 +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: 13.0
X-Bugzilla-Keywords: ice-on-valid-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: hubicka at ucw dot cz
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: jason at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 13.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-108887-4-xv1FRz4rPW@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-108887-4@http.gcc.gnu.org/bugzilla/>
References: <bug-108887-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
List-Id: <gcc-bugs.sourceware.org>

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

--- Comment #8 from Jan Hubicka <hubicka at ucw dot cz> ---
> Also, reset() is only defined in cgraph_node, and I need it to work on bo=
th
> functions and variables.
Aha, this is a good point. I forgot that.  I will make reset() working
on symbols in general.  I think it is cleanest way ahead.
>=20
> Clearing n->type seems to confuse things that expect all symbols to be ei=
ther
> function or variable.
Yes, SYMBOL is not really used in symbol table except for temporary
situations during the construction. So I would indeed expect things to
be confused.=