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 D8D5B3858CD1; Tue,  2 Apr 2024 22:04:01 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D8D5B3858CD1
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1712095441;
	bh=NMTojkQo618p1DlmEUsEY/PKs57j6KTtptRvbByiUOw=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=NzJGScf8WeLNZqVbdDvCpTRQFgUJfoTXBoYCKKODlUYAoqWT9gj0emxlZ504ICuts
	 p9KxCtxwwk+hs04+bAXkacMJ71fBc37q+1UPMpxaG7K/NwdO4gxuoH6QE+7+lT6hHN
	 NGwSXr6Mb1nLzBq/we7voHL9xY/s9xqc+V+NJ1y0=
From: "jason at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/111067] g++.dg/opt/icf{1,2,3}.C tests fail on darwin
Date: Tue, 02 Apr 2024 22:04:01 +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: 14.0
X-Bugzilla-Keywords: testsuite-fail
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jason at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-111067-4-yt7Ro3GoTk@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-111067-4@http.gcc.gnu.org/bugzilla/>
References: <bug-111067-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=3D111067

--- Comment #10 from Jason Merrill <jason at gcc dot gnu.org> ---
(In reply to Jonathan Wakely from comment #8)
> (In reply to Iain Sandoe from comment #7)
> > So I am actually asking if the extension actually has any useful meanin=
g?
>=20
> For non-darwin, yes, it requests the storage of two initializer lists to =
be
> merged (see the commit msg for r14-1500-g4d935f52b0d5c0).

Though that doesn't involve the attribute, and promoting init-lists to stat=
ic
should work fine on darwin.

(In reply to Jonathan Wakely from comment #6)
> The question then is whether the attribute is supposed to be a non-binding
> request or not.
>=20
> If it's a non-binding request then the test should be adjusted/unsupported
> for this target.

It is a non-binding request. And yes, if this optimization is problematic on
darwin, we should adjust the test.=