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 4371938582A3; Wed, 20 Mar 2024 12:07:09 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4371938582A3
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1710936429;
	bh=EvEzIhOKEpCySPQggaObvNqMPg3rv8BIQEZlvJnSYLQ=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=KMya8clCuA3de5Np4N2NxNDeKEFBHH/siC9jJvARsJyE6qTsYO83lnB+XRuJgjuFc
	 M4LbxFb37RGtZ2aaYv44Lpr/LVuxnJoV2T5Pw9gFWMzl00xWCOyDje+GPPQP+1jwD9
	 dgmOaY5u01efnSYdo6WqNoO9PY/B509WbmUKXwjI=
From: "redi at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug libstdc++/28277] __builtin_alloca with no limit in libstdc++
Date: Wed, 20 Mar 2024 12:07:08 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: libstdc++
X-Bugzilla-Version: 4.2.0
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: enhancement
X-Bugzilla-Who: redi 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-28277-4-dETmHKxGKi@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-28277-4@http.gcc.gnu.org/bugzilla/>
References: <bug-28277-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=3D28277

--- Comment #22 from Jonathan Wakely <redi at gcc dot gnu.org> ---
(In reply to Jonathan Wakely from comment #21)
> (In reply to Andrew Pinski from comment #20)
> > src/c++11/snprintf_lite.cc (__throw_insufficient_space)
>=20
> Another internal impl detail, but this one is not even visible to users. =
No
> declaration in headers, not exported from the shared lib. This one uses
> alloca for the same size as __throw_out_of_range_fmt + 104 bytes, but we'=
ve
> already done the first alloca, so it's (strlen(fmt) + 512) * 2 + 104.

There's also __concat_size_t in that file, but it only uses alloca for
3*sizeof(size_t), and that is unnecessary and should be replaced with
std::to_chars.=