Date: Tue, 14 Dec 2021 12:05:52 -0800 From: Steve Kargl <sgk@troutmask.apl.washington.edu> To: enh via freebsd-numerics <freebsd-numerics@freebsd.org> Cc: mmel@freebsd.org Subject: Re: s_sincosl.c: remove unused variable Message-ID: <20211214200552.GA49922@troutmask.apl.washington.edu> In-Reply-To: <CAJgzZoqrBgxto5k5eeJ1nuPu=BHLk1MfE74KN4rQh%2BghV8ehyQ@mail.gmail.com> References: <CAJgzZoqrBgxto5k5eeJ1nuPu=BHLk1MfE74KN4rQh%2BghV8ehyQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
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. -- Steve
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20211214200552.GA49922>