Re: s_sincosl.c: remove unused variable
- In reply to: Dimitry Andric : "Re: s_sincosl.c: remove unused variable"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 15 Dec 2021 01:45:36 UTC
heh, no worries --- thanks to everyone here for maintaining the world's most-used (Android *and* iOS/macOS!) libm :-) On Tue, Dec 14, 2021 at 1:54 PM Dimitry Andric <dim@freebsd.org> wrote: > On 14 Dec 2021, at 21:05, Steve Kargl <sgk@troutmask.apl.washington.edu> > wrote: > > > > On Tue, Dec 14, 2021 at 11:06:53AM -0800, enh via freebsd-numerics wrote: > >> Newer versions of clang complain about this, and it seems genuinely > unneeded. > >> > >> It was in the original commit, and judging from that I think it's just > >> a copy & paste leftover from inlinine sinl() and cosl() --- the < > >> M_PI_4 case in sinl() needs the original sign. > > > > Yep. sgn is unneeded. Sorry about that. > > Thanks, committed in: > > https://cgit.freebsd.org/src/commit/?id=20d425842a4f80f9efc1de3f7a3b043435fe4b39 > > -Dimitry > > >