riscv: fmax/fmin sNaN fix

DJ Delorie dj@redhat.com
Thu Feb 22 04:30:00 GMT 2018


"Carlos O'Donell" <carlos@redhat.com> writes:
> Patch review includes review of the commit messages for factual accuracy.

Please review the original changelog entry then ;-)

>	* sysdeps/riscv/rvd/s_fmax.c (__fmax): Handle sNaNs correctly.
> 	* sysdeps/riscv/rvd/s_fmin.c (__fmin): Likewise.
> 	* sysdeps/riscv/rvf/s_fmaxf.c (__fmaxf): Likewise.
> 	* sysdeps/riscv/rvf/s_fminf.c (__fminf): Likewise.

That's actually functional, and doesn't mention anything about specs,
should be sufficient.

Original author: Andrew Waterman <andrew@sifive.com>

(plus or minus the "maintainers have automatic consensus" rules, of
course ;)



More information about the Libc-alpha mailing list