From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 437F83870867; Mon, 29 Jun 2020 19:28:38 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 437F83870867 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1593458918; bh=nJtDr9YabeBN8bIwovlAv9YL2rM+Pc6CJ2xQDty8Xgw=; h=From:To:Subject:Date:In-Reply-To:References:From; b=P7CTdzwCjXQhG8C1Ou5WlQuZWMXBe8xWRfe9gJSqDmds2O9Guu6hCCDTu3N6VQrZq ivIMWa759fgH7ZllmXBy7Wd5PdRrJfojkeOFVS4P4jPVbzuHwxm3LVwd2z6o16Ofmr op971b2E+QaDFb+B3lDyDWTl7/NWvKFCBFrM3vzg= From: "jakub at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/95985] [11 Regression] ICE: tree check: expected tree_list, have error_mark in common_handle_aligned_attribute, at c-family/c-attribs.c:1980 Date: Mon, 29 Jun 2020 19:28:38 +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: error-recovery, ice-on-invalid-code X-Bugzilla-Severity: normal X-Bugzilla-Who: jakub at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P4 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 11.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: priority keywords Message-ID: In-Reply-To: References: 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 29 Jun 2020 19:28:38 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D95985 Jakub Jelinek changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P3 |P4 Keywords| |error-recovery --- Comment #2 from Jakub Jelinek --- While we could do something like: --- c-attribs.c.jj 2020-06-10 14:58:17.422495938 +0200 +++ c-attribs.c 2020-06-29 21:25:58.300888506 +0200 @@ -1975,7 +1975,9 @@ common_handle_aligned_attribute (tree *n yet. */ tree last_decl =3D node[1] ? node[1] : *node; - if (args) + if (args =3D=3D error_mark_node) + align_expr =3D args; + else if (args) { align_expr =3D TREE_VALUE (args); if (align_expr && TREE_CODE (align_expr) !=3D IDENTIFIER_NODE I don't think it scales well, because we have hundreds of *handle_*attribute functions and all of them assume that args is either NULL or a TREE_LIST and changing them doesn't look right.=