RFA: Add ! LDBL_EQ_DBL implementations of cosl, sinl and tanl

Corinna Vinschen vinschen@redhat.com
Tue Mar 3 14:44:00 GMT 2015


On Mar  3 11:23, Nicholas Clifton wrote:
> Hi Corinna,
> 
> >That's right, a math.h patch is missing.
> 
> Snafu - I had patched the file, but forgot to include it in the submission.
> :-(
> 
> >Setting errno should be easy to add:
> >
> >- cosl, sinl and tanl just need an extra check for Inf to set errno to
> >   EDOM.
> >
> >- tanl needs an extra check if the result of a non-Inf arg is Inf to
> >   return ERANGE.
> >
> >- fabsl doesn't define errors>
> >
> >- floorl doesn't produce errors (see Linux man page),
> >
> >Nick, any problem to add this?
> 
> I can do this, but... sleeping on it last night, I realised that I had not
> checked all of the code paths in the new functions.  When I checked them
> this morning I discovered that they were broken.  Ie sinl(), cosl() and
> tanl() all return incorrect values for certain, large, input values.  I
> tried to investigate, but the maths involved is beyond me, so I am going to
> have to shelve this patch until I have more time.  Sorry.

Ok, no worries.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20150303/4382fe6a/attachment.sig>


More information about the Newlib mailing list