It's now live at https://hg.openjdk.java.net/jdk/jdk/rev/9e54ea7d9cd9.
Thanks, Max > On Feb 5, 2020, at 8:53 PM, Patrick Zhang OS <patr...@os.amperecomputing.com> > wrote: > > Hi, > > Please help review this quick patch in order to resolve the libj2gss link > errors caused by GCC10 default option -fno-common. More details, please see > JDK-8238388 and its related JDK-8235903 [1]. > > Weijun, I saw your comments in the ticket, thanks. It is exactly the same way > I am updating the code. In addition, could you please sponsor this patch > (once approved)? Thanks. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8238388 > Webrev: http://cr.openjdk.java.net/~qpzhang/8238388/webrev.01/ > Test: ran jtreg tier1, jdk built with GCC4.8.5, GCC9, and GCC10, no > regression found > > [1] https://bugs.openjdk.java.net/browse/JDK-8235903 GCC default -fno-common > exposes "multiple definition" link errors > > Regards > Patrick
