This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: Add _Float128 function aliases [committed]
- From: Szabolcs Nagy <szabolcs dot nagy at arm dot com>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: nd at arm dot com, libc-alpha at sourceware dot org
- Date: Tue, 09 Jan 2018 17:15:10 +0000
- Subject: Re: Add _Float128 function aliases [committed]
- Authentication-results: sourceware.org; auth=none
- Authentication-results: spf=none (sender IP is ) smtp.mailfrom=Szabolcs dot Nagy at arm dot com;
- Nodisclaimer: True
- References: <alpine.DEB.2.20.1710181737470.28577@digraph.polyomino.org.uk> <5A54E211.8080702@arm.com> <alpine.DEB.2.20.1801091642050.9564@digraph.polyomino.org.uk>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
On 09/01/18 16:47, Joseph Myers wrote:
> On Tue, 9 Jan 2018, Szabolcs Nagy wrote:
>
>> note that this typedef in a public header does not work
>> on compilers that support _Float128 but not >=gcc-7
>
> It's entirely appropriate to add support for other compilers in
> bits/floatn.h headers (there are several bits/floatn.h variants in glibc -
> if it's a matter of "clang version N supports _Float128 on this set of
> configurations", the headers used by those configurations are the ones
> appropriate to update), where those compilers have some form of _FloatN /
> _FloatNx / __float128 support that could reasonably be used with installed
> glibc headers.
>
i can provide a workaround in aarch64 target specific
headers, but i think this is should be fixed in
sysdeps/ieee754/ldbl-128/bits/floatn.h
maybe check for __FLT128_MANT_DIG==113 instead of gcc-7?
clang defines this macro too.