fmod nan_mix usage
Bruce Evans
brde at optusnet.com.au
Mon Jul 23 21:41:22 UTC 2018
On Mon, 23 Jul 2018, Steve Kargl wrote:
> On Mon, Jul 23, 2018 at 11:28:08AM -0700, enh via freebsd-numerics wrote:
>> the recent change from
>>
>> return (x*y)/(x*y);
>>
>> to
>>
>> return nan_mix(x, y)/nan_mix(x, y);
>>
>> in e_fmod.c broke some of our unit tests. for example, fmod(3.f, 0.f) in
>> one of the VM tests.
>>
>> bionic/tests/math_test.cpp:(784) Failure in test
>> math_h_force_long_double.fmod
>> Value of: isnan(fmod(3.0, 0.0))
>> Actual: false
>> Expected: true
>
> Can you share the code for the relevant tests?
> This simple program gives the expected results
> on amd64.
>
> #include <math.h>
> #include <stdio.h>
>
> int
> main(void)
> {
> printf("%e %d\n", fmodf(3.f, 0.f), isnan(fmodf(3.f, 0.f)));
> printf("%le %d\n", fmod(3.0, 0.0), isnan(fmod(3.0, 0.0)));
> printf("%Le %d\n", fmodl(3.L, 0.L), isnan(fmodl(3.L, 0.L)));
> return 0;
> }
>
> % cc -o z -O a.c -lm && ./z
> nan 1
> nan 1
> nan 1
clang normally evaluates this at compile, so it doesn't test the libary.
This is arguably a bug in clang, since it doesn't set the exception flags.
#pragma FENV_ACCESS should control this, but it is hard to use and rarely
works.
The test data needs to be non-literal and perhaps even volatile to prevent
the compiler evaluating it at compile time.
Bruce
More information about the freebsd-numerics
mailing list