From nobody Mon Jun 14 22:15:42 2021 X-Original-To: freebsd-current@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 14B2E11D3E4F for ; Mon, 14 Jun 2021 22:15:48 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4G3m435xTmz3KvG; Mon, 14 Jun 2021 22:15:47 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from tensor.andric.com (tensor.andric.com [87.251.56.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "tensor.andric.com", Issuer "R3" (verified OK)) (Authenticated sender: dim) by smtp.freebsd.org (Postfix) with ESMTPSA id 9425A21719; Mon, 14 Jun 2021 22:15:47 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from smtpclient.apple (unknown [IPv6:2001:470:7a58:0:9062:5662:3b4e:5bd]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by tensor.andric.com (Postfix) with ESMTPSA id 62357B9A9; Tue, 15 Jun 2021 00:15:46 +0200 (CEST) From: Dimitry Andric Message-Id: Content-Type: multipart/signed; boundary="Apple-Mail=_0B3F64FC-7FAB-40DD-8FD1-C5BD97948E5C"; protocol="application/pgp-signature"; micalg=pgp-sha1 List-Id: Discussions about the use of FreeBSD-current List-Archive: https://lists.freebsd.org/archives/freebsd-current List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-current@freebsd.org Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.100.0.2.22\)) Subject: Re: git: 790a6be5a169 - main - Export various 128 bit long double functions from libgcc_s.so.1 Date: Tue, 15 Jun 2021 00:15:42 +0200 In-Reply-To: <38C7A31A-CB21-4C3C-98D0-38F91EBC35D3@yahoo.com> Cc: freebsd-current To: Mark Millard References: <38C7A31A-CB21-4C3C-98D0-38F91EBC35D3.ref@yahoo.com> <38C7A31A-CB21-4C3C-98D0-38F91EBC35D3@yahoo.com> X-Mailer: Apple Mail (2.3654.100.0.2.22) X-ThisMailContainsUnwantedMimeParts: N --Apple-Mail=_0B3F64FC-7FAB-40DD-8FD1-C5BD97948E5C Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=us-ascii On 14 Jun 2021, at 22:38, Mark Millard wrote: >=20 > Dimitry Andric dim at FreeBSD.org wrote on > Mon Jun 14 19:17:40 UTC 2021 : >=20 >> The branch main has been updated by dim: >>=20 >> URL: >> = https://cgit.FreeBSD.org/src/commit/?id=3D790a6be5a1699291c6da87871426d0c5= 6dedcc89 >>=20 >>=20 >> commit 790a6be5a1699291c6da87871426d0c56dedcc89 >> Author: Dimitry Andric >> AuthorDate: 2021-06-14 19:17:05 +0000 >> Commit: Dimitry Andric >> CommitDate: 2021-06-14 19:17:05 +0000 >>=20 >> Export various 128 bit long double functions from libgcc_s.so.1 >>=20 >> These were already compiled for some time on aarch64 and riscv, by >> including lib/libcompiler_rt/Makefile.inc, but never exported in = the >> shared library. Since gcc exports these under version GCC_4.6.0, = we do >> the same. >>=20 >> This review should replace D11482 for now. For e.g. amd64 more = work is >> still to be done, as compiler-rt does not seem to support 128 bit = long >> double math for that architecture. >=20 > Given that "man arch" explicitly reports: >=20 > Architecture void * long double time_t > . . . > amd64 8 16 8 > . . . >=20 > and also: >=20 > Architecture float, double long double > . . . > amd64 hard hard, 80 bit > . . . Yeah this is a bit confusing, since padding is never mentioned. > (so there is guaranteed pad involved), would amd64 having 128 > bit long doubles (no pad by default?) be an FreeBSD ABI change, > possibly needing a discussion on freebsd-arch and > freebsd-numerics ? (Or may be it happened and I just do not > remember. Or may be it would take special enabling of the > alternate ABI's use and not be a default configuration.) >=20 > (i386 only has "12", not "16", for its "80 bit", so 96 bits: > less pad.) There's nothing to worry about, the "tf" variants in this update are only enabled for aarch64 and riscv; see lib/libcompiler_rt/Makefile.inc: # # 128-bit quad precision long double support, # only used on some architectures. # .if ${MACHINE_CPUARCH} =3D=3D "aarch64" || ${MACHINE_CPUARCH} =3D=3D = "riscv" SRCF+=3D addtf3 SRCF+=3D comparetf2 SRCF+=3D divtf3 SRCF+=3D extenddftf2 SRCF+=3D extendsftf2 SRCF+=3D fixtfdi SRCF+=3D fixtfsi SRCF+=3D fixtfti SRCF+=3D fixunstfdi SRCF+=3D fixunstfsi SRCF+=3D fixunstfti SRCF+=3D floatsitf SRCF+=3D floattitf SRCF+=3D floatunsitf SRCF+=3D floatuntitf SRCF+=3D multf3 SRCF+=3D subtf3 SRCF+=3D trunctfdf2 SRCF+=3D trunctfsf2 .endif I'm sorry to have sowed confusion here, as the commit message was mostly copy/pasted from the Phabricator review. Please consider the amd64 sentence to be 'cancelled'. :) -Dimitry --Apple-Mail=_0B3F64FC-7FAB-40DD-8FD1-C5BD97948E5C Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.2 iF0EARECAB0WIQR6tGLSzjX8bUI5T82wXqMKLiCWowUCYMfVDgAKCRCwXqMKLiCW o/1LAKCeF+9QXubklasDw4naLdgG25jesgCg7VFiFc4jZt9fZWpp2D+bUDGsWXA= =azeR -----END PGP SIGNATURE----- --Apple-Mail=_0B3F64FC-7FAB-40DD-8FD1-C5BD97948E5C--