From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id E1575385800A; Wed, 29 Mar 2023 15:00:40 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E1575385800A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1680102040; bh=+p3LAou+3meOmOYidNSiNyJM9WX10ecnkCjphRXPmGQ=; h=From:To:Subject:Date:In-Reply-To:References:From; b=wTwbQzVBOctJ80BOTylcgBAGDZek8WGedgzqFDDTV8OTRzuQgZ11RE2XKGwee78kq ETqRDWOwT00ep14p4yC3vYcu41gVhcL02ZDXz3Mjl+r+vc94qhoxE2b6LYxO9Fo5KP SmUw0je2NezBqvtJNEw9rzabbdWkjrc5g9wwPZao= From: "ro at CeBiTec dot Uni-Bielefeld.DE" To: gcc-bugs@gcc.gnu.org Subject: [Bug d/109231] [13 regression] Comparison failure in libphobos/libdruntime/rt/util/typeinfo.o Date: Wed, 29 Mar 2023 15:00:40 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: d X-Bugzilla-Version: 13.0 X-Bugzilla-Keywords: build X-Bugzilla-Severity: normal X-Bugzilla-Who: ro at CeBiTec dot Uni-Bielefeld.DE X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: ibuclaw at gdcproject dot org X-Bugzilla-Target-Milestone: 13.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: 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 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D109231 --- Comment #24 from ro at CeBiTec dot Uni-Bielefeld.DE --- > --- Comment #23 from Jakub Jelinek --- [...] > Perhaps try to undo my patch in a different way, like > --- gcc/tree-inline.cc 2023-03-17 18:59:50.226199917 +0100 > +++ gcc/tree-inline.cc 2023-03-29 12:47:21.546947442 +0200 > @@ -2785,7 +2785,7 @@ initialize_cfun (tree new_fndecl, tree c > gimple_register_cfg_hooks (); > > /* Get clean struct function. */ > - push_struct_function (new_fndecl, true); > + push_struct_function (new_fndecl, false); > targetm.target_option.relayout_function (new_fndecl); > > /* We will rebuild these, so just sanity check that they are empty. */ > or > --- gcc/tree-inline.cc 2023-03-17 18:59:50.226199917 +0100 > +++ gcc/tree-inline.cc 2023-03-29 12:49:16.580255668 +0200 > @@ -2786,7 +2786,11 @@ initialize_cfun (tree new_fndecl, tree c > > /* Get clean struct function. */ > push_struct_function (new_fndecl, true); > + relayout_decl (DECL_RESULT (new_fndecl)); > + for (tree parm =3D DECL_ARGUMENTS (new_fndecl); parm; parm =3D DECL_CH= AIN > (parm)) > + relayout_decl (parm); > targetm.target_option.relayout_function (new_fndecl); > + aggregate_value_p (DECL_RESULT (new_fndecl), new_fndecl); > > /* We will rebuild these, so just sanity check that they are empty. */ > gcc_assert (VALUE_HISTOGRAMS (cfun) =3D=3D NULL); > and see if that changes anything? Of course both of those patches break = the > PR105554 > again. Or if the latter helps, try to comment out the different parts of= it > too. So far, I've tried both variants and in each case, the comparison failure is gone. > Seems there was some valgrind for SPARC Solaris out of tree, but can't fi= nd it > anymore... There was one back in the 2014-2017 timeframe, but the sources lived on bitbucket and are gone, apparently. The author (Ivo Raisr) since left Oracle and works for RedHat/Prague AFAICT. However, reviving that port even if the sources were available would be a major feat.=