Don't make local variables static in ldbl-96 j1l.

The ldbl-96 implementation of j1l has some function-local variables
that are declared static for no apparent reason (this dates back to
the first addition of that file).

Any vaguely recent compiler, probably including any that are supported
for building glibc, optimizes away the "static" here, as the values of
the variables on entry to the function are dead.  So there is not
actually a user-visible bug here at present (but with any compilers
that didn't optimize away the static at all, possibly building with
less or no optimization, so that the function stored intermediate
values to and then loaded them from the variables, there would have
been a thread-safety issue).  But the "static" clearly doesn't belong
there and might potentially make things unsafe were compilation
without optimization to be supported in future, so this patch removes
it.

Tested for x86_64.

	* sysdeps/ieee754/ldbl-96/e_j1l.c (qone): Don't make local
	variables static.

diff --git a/ChangeLog b/ChangeLog
index 7850a15f63bec0baae898693e00eb00f56fa7507..eda9c0ca82974abf6ee0286f652a35ddc49266c0 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2017-12-06  Joseph Myers  <joseph@codesourcery.com>
 
+       * sysdeps/ieee754/ldbl-96/e_j1l.c (qone): Don't make local
+       variables static.
+
        * sysdeps/ieee754/ldbl-128/e_j0l.c (Y0_2N): Make const.
        (Y0_2D): Likewise.
        * sysdeps/ieee754/ldbl-128/e_j1l.c (Y0_2N): Likewise.

diff --git a/sysdeps/ieee754/ldbl-96/e_j1l.c b/sysdeps/ieee754/ldbl-96/e_j1l.c
index a8a9902db0e8aec31f40aa154976121493933e6d..4f07b0b3272dc85a419a35bdaad0ff7f5c1ed54b 100644 (file)
--- a/sysdeps/ieee754/ldbl-96/e_j1l.c
+++ b/sysdeps/ieee754/ldbl-96/e_j1l.c
@@ -507,7 +507,7 @@ static long double
 qone (long double x)
 {
   const long double *p, *q;
-  static long double s, r, z;
+  long double s, r, z;
   int32_t ix;
   uint32_t se, i0, i1;