This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2 07/15] RISC-V: RV32F Support
- From: Palmer Dabbelt <palmer at dabbelt dot com>
- To: joseph at codesourcery dot com
- Cc: libc-alpha at sourceware dot org, Andrew Waterman <andrew at sifive dot com>, Darius Rad <darius at bluespec dot com>, dj at redhat dot com
- Date: Sat, 23 Dec 2017 16:37:12 -0800 (PST)
- Subject: Re: [PATCH v2 07/15] RISC-V: RV32F Support
- Authentication-results: sourceware.org; auth=none
On Wed, 20 Dec 2017 09:01:28 PST (-0800), joseph@codesourcery.com wrote:
On Tue, 19 Dec 2017, Palmer Dabbelt wrote:
+ int nan = isnanf (x);
I don't think any libm code should be using the legacy X/Open
type-specific classification functions such as isnanf. Use the C99
type-generic macros such as isnan instead.
+ int nan = isnanf (x);
Likewise, and again throughout this patch.
If I understand correctly, these are just
* fpclassify
* isnan
* isfinate
* isinf
* isnormal
* signbit
and there's no macros for
* fabs
* copysign
If so, I believe I've fixed them all.
Thanks!