]> sourceware.org Git - glibc.git/commit
Don't make local variables static in ldbl-96 j1l.
authorJoseph Myers <joseph@codesourcery.com>
Wed, 6 Dec 2017 15:19:06 +0000 (15:19 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Wed, 6 Dec 2017 15:19:06 +0000 (15:19 +0000)
commit1dbe6f64ab982a1a3258948e9347b5bca76790f3
tree339a780ad228552146a5dd42a6bc0d0ace5ed909
parent53994f12633b863eb833fc3b1f6971ae8de61ff8
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.
ChangeLog
sysdeps/ieee754/ldbl-96/e_j1l.c
This page took 0.042459 seconds and 5 git commands to generate.