Date: Tue, 14 Dec 2021 11:06:53 -0800 From: enh via freebsd-numerics <freebsd-numerics@freebsd.org> To: freebsd-numerics@freebsd.org Cc: mmel@freebsd.org Subject: s_sincosl.c: remove unused variable Message-ID: <CAJgzZoqrBgxto5k5eeJ1nuPu=BHLk1MfE74KN4rQh%2BghV8ehyQ@mail.gmail.com>
next in thread | raw e-mail | index | archive | help
--00000000000019b5ee05d31fe538
Content-Type: text/plain; charset="UTF-8"
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.
(I've cc:ed the original author.)
diff --git a/lib/msun/src/s_sincosl.c b/lib/msun/src/s_sincosl.c
index aef36c2320b..3dd34575f89 100644
--- a/lib/msun/src/s_sincosl.c
+++ b/lib/msun/src/s_sincosl.c
@@ -50,11 +50,10 @@ void
sincosl(long double x, long double *sn, long double *cs)
{
union IEEEl2bits z;
- int e0, sgn;
+ int e0;
long double y[2];
z.e = x;
- sgn = z.bits.sign;
z.bits.sign = 0;
ENTERV();
--00000000000019b5ee05d31fe538
Content-Type: text/plain; charset="US-ASCII"; name="patch.txt"
Content-Disposition: attachment; filename="patch.txt"
Content-Transfer-Encoding: base64
Content-ID: <f_kx6h0hi00>
X-Attachment-Id: f_kx6h0hi00
ZGlmZiAtLWdpdCBhL2xpYi9tc3VuL3NyYy9zX3NpbmNvc2wuYyBiL2xpYi9tc3VuL3NyYy9zX3Np
bmNvc2wuYwppbmRleCBhZWYzNmMyMzIwYi4uM2RkMzQ1NzVmODkgMTAwNjQ0Ci0tLSBhL2xpYi9t
c3VuL3NyYy9zX3NpbmNvc2wuYworKysgYi9saWIvbXN1bi9zcmMvc19zaW5jb3NsLmMKQEAgLTUw
LDExICs1MCwxMCBAQCB2b2lkCiBzaW5jb3NsKGxvbmcgZG91YmxlIHgsIGxvbmcgZG91YmxlICpz
biwgbG9uZyBkb3VibGUgKmNzKQogewogCXVuaW9uIElFRUVsMmJpdHMgejsKLQlpbnQgZTAsIHNn
bjsKKwlpbnQgZTA7CiAJbG9uZyBkb3VibGUgeVsyXTsKIAogCXouZSA9IHg7Ci0Jc2duID0gei5i
aXRzLnNpZ247CiAJei5iaXRzLnNpZ24gPSAwOwogCiAJRU5URVJWKCk7Cg==
--00000000000019b5ee05d31fe538--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJgzZoqrBgxto5k5eeJ1nuPu=BHLk1MfE74KN4rQh%2BghV8ehyQ>