* Florian Weimer: > * Florian Weimer: > >> __kernel_standard in src/java.base/share/native/libfdlibm/k_standard.c >> is built for _IEEE_LIBM targets as well, but it appears superfluous >> there. >> >> In noticed this because GCC 10 flags an uninitialized variable in this >> code: >> >> …/src/java.base/share/native/libfdlibm/k_standard.c: In function >> '__j__kernel_standard': >> …/src/java.base/share/native/libfdlibm/k_standard.c:743:19: error: >> 'exc.retval' may be used uninitialized in this function >> [-Werror=maybe-uninitialized] >> 743 | return exc.retval; >> | ~~~^~~~~~~ >> >> Rather than debating whether this warning is reasonableor not, I would >> suggest to #ifdef out this code for _IEEE_LIBM targets (are there any >> else left?). > > I think the file should just be removed. > > Bug: <https://bugs.openjdk.java.net/browse/JDK-8234362> > Webrev: <http://cr.openjdk.java.net/~fweimer/8234362/webrev.01/> > > As usual, I'll need a sponsor for this.
Is there anything I can do to make this patch more palatable? Thanks, Florian
