CVS commit: src/lib/libm/src

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

Module Name:    src
Committed By:   riastradh
Date:           Wed Oct 30 16:57:16 UTC 2024

Modified Files:
        src/lib/libm/src: s_nearbyint.c

Log Message:
nearbyint(3), nearbyintf(3): Use rint(3), rintf(3), not rintl(3).

This macro was mistakenly changed to use rintl(3) for all three
functions, instead of using the macro parameter as originally
intended.  Put it back the right way, and rename the paramter
`rintfn' to avoid confusion in the future.

(It is not strictly wrong to use rintl(3) for nearbyint(3) and
nearbyintf(3), because the input can always be converted to long
double without losing information, and once rounded to an integer,
the output undergoes no further rounding back into float or double --
if the input was an integer already it is unchanged, and if the input
was not then rounding it to an integer can't use more precision than
was already needed for the input.  But it might be suboptimal.)

Should fix failures in nearbyint/nearbyintf tests on sparc64 because
rintl is broken (nearbyintl is still broken because rintl is still
broken, of course, to be diagnosed).


To generate a diff of this commit:
cvs rdiff -u -r1.5 -r1.6 src/lib/libm/src/s_nearbyint.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.