[PATCH] Fix math functions triggering invalid exceptions
Ryan S. Arnold
ryan.arnold@gmail.com
Wed Feb 15 21:42:00 GMT 2012
On Wed, Feb 15, 2012 at 3:29 PM, Aurelien Jarno <aurelien@aurel32.net> wrote:
> On Wed, Feb 15, 2012 at 08:55:52PM +0000, Joseph S. Myers wrote:
...
>> So I think you should repost with those fixes, and then commit if noone
>> else spots any problems and if it's been tested on both an affected
>> platform and one that didn't previously have the issue you report.
>>
>
> Thanks for the review. Here is the new patch.
>
>
> 2012-02-15 Aurelien Jarno <aurelien@aurel32.net>
>
> * math/w_acos.c: Use non-signaling floating-point comparisons.
> * math/w_acosf.c: Likewise.
> * math/w_acosh.c: Likewise.
> * math/w_acoshf.c: Likewise.
> * math/w_acoshl.c: Likewise.
> * math/w_acosl.c: Likewise.
> * math/w_asin.c: Likewise.
> * math/w_asinf.c: Likewise.
> * math/w_asinl.c: Likewise.
> * math/w_atanh.c: Likewise.
> * math/w_atanhf.c: Likewise.
> * math/w_atanhl.c: Likewise.
> * math/w_exp2.c: Likewise.
> * math/w_exp2f.c: Likewise.
> * math/w_exp2l.c: Likewise.
> * math/w_j0.c: Likewise.
> * math/w_j0f.c: Likewise.
> * math/w_j0l.c: Likewise.
> * math/w_j1.c: Likewise.
> * math/w_j1f.c: Likewise.
> * math/w_j1l.c: Likewise.
> * math/w_jn.c: Likewise.
> * math/w_jnf.c: Likewise.
> * math/w_log.c: Likewise.
> * math/w_log10.c: Likewise.
> * math/w_log10f.c: Likewise.
> * math/w_log10l.c: Likewise.
> * math/w_log2.c: Likewise.
> * math/w_log2f.c: Likewise.
> * math/w_log2l.c: Likewise.
> * math/w_logf.c: Likewise.
> * math/w_logl.c: Likewise.
> * math/w_sqrt.c: Likewise.
> * math/w_sqrtf.c: Likewise.
> * math/w_sqrtl.c: Likewise.
> * sysdeps/ieee754/dbl-64/e_atanh.c: Likewise.
> * sysdeps/ieee754/dbl-64/w_exp.c: Likewise.
> * sysdeps/ieee754/flt-32/e_atanhf.c: Likewise.
> * sysdeps/ieee754/flt-32/w_expf.c: Likewise.
> * sysdeps/ieee754/ldbl-96/w_expl.c: Likewise.
Please hold up on checking this in. We'd like some time to
investigate the performance implications of these additional
comparisons which very well may be unnecessary on Power.
Ryan S. Arnold
More information about the Libc-alpha
mailing list