RFR JDK-8209786: gcc 7.3 compiler errors on zLinux (original) (raw)

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Fri Aug 31 06:50:23 UTC 2018

Hi Leonard,

Whom should I add as reviewers? (Besides me :))

Best regards, Goetz.

-----Original Message----- From: Andrew Leonard <andrewmleonard at uk.ibm.com> Sent: Donnerstag, 30. August 2018 17:02 To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com> Cc: Brian Burkhalter <brian.burkhalter at oracle.com>; build-dev (build- dev at openjdk.java.net) <build-dev at openjdk.java.net>; core-libs- dev at openjdk.java.net Subject: RE: RFR JDK-8209786: gcc 7.3 compiler errors on zLinux

Thanks Goetz, I've created an hg export below.. I've not used jdk-submit, i've tested it locally on xLinux and zLinux. Cheers Andrew # HG changeset patch # User aleonard # Date 1535641094 -3600 # Thu Aug 30 15:58:14 2018 +0100 # Node ID 592a8ad8d4c16287c316d018a0a402148720718b # Parent 1ddd1ec044311512c55643bed641859e78b9d25e 8209786: disable gcc warnings for libmlib & libfdlibm to enable gcc 7.3 on zLinux diff -r 1ddd1ec04431 -r 592a8ad8d4c1 make/lib/Awt2dLibraries.gmk --- a/make/lib/Awt2dLibraries.gmk Thu Aug 30 09:08:23 2018 -0400 +++ b/make/lib/Awt2dLibraries.gmk Thu Aug 30 15:58:14 2018 +0100 @@ -55,6 +55,7 @@ _OPTIMIZATION := HIGHEST, _ _CFLAGS := $(CFLAGSJDKLIB) _ _$(BUILDLIBMLIBCFLAGS), _ _+ DISABLEDWARNINGSgcc := shift-negative-value, _ _LDFLAGS := $(LDFLAGSJDKLIB) _ _$(call SETSHAREDLIBRARYORIGIN), _ _LIBS := $(JDKLIBLIBS), _ diff -r 1ddd1ec04431 -r 592a8ad8d4c1 make/lib/CoreLibraries.gmk --- a/make/lib/CoreLibraries.gmk Thu Aug 30 09:08:23 2018 -0400 +++ b/make/lib/CoreLibraries.gmk Thu Aug 30 15:58:14 2018 +0100 @@ -68,7 +68,7 @@ _CFLAGSlinuxppc64le := -ffp-contract=off, _ _CFLAGSlinuxs390x := -ffp-contract=off, _ _CFLAGSlinuxaarch64 := -ffp-contract=off, _ _- DISABLEDWARNINGSgcc := sign-compare misleading-indentation, _ + DISABLEDWARNINGSgcc := sign-compare misleading-indentation _array-bounds, _ _DISABLEDWARNINGSmicrosoft := 4146 4244 4018, _ _ARFLAGS := $(ARFLAGS), _ _OBJECTDIR := (SUPPORTOUTPUTDIR)/native/(SUPPORTOUTPUTDIR)/native/(SUPPORTOUTPUTDIR)/native/(MODULE)/libfdlibm, _

Andrew Leonard Java Runtimes Development IBM Hursley IBM United Kingdom Ltd Phone internal: 245913, external: 01962 815913 internet email: andrewmleonard at uk.ibm.com From: "Lindenmaier, Goetz" <goetz.lindenmaier at sap.com> To: Andrew Leonard <andrewmleonard at uk.ibm.com>, Brian Burkhalter <brian.burkhalter at oracle.com> Cc: "core-libs-dev at openjdk.java.net" <core-libs-_ _dev at openjdk.java.net>, "build-dev (build-dev at openjdk.java.net)" <build-_ _dev at openjdk.java.net> Date: 30/08/2018 15:36 Subject: RE: RFR JDK-8209786: gcc 7.3 compiler errors on zLinux

Hi Leonard, the change looks good to me. I'll test it tonight, to make sure it runs with our compilers. Did you run it through jdk-submit? If you supply a patch with all the changeset information (like from hg export) that jchecks fine, I'll sponsor this for you. Best regards, Goetz. > -----Original Message----- > From: core-libs-dev <core-libs-dev-bounces at openjdk.java.net> On Behalf > Of Andrew Leonard > Sent: Donnerstag, 30. August 2018 14:19 > To: Brian Burkhalter <brian.burkhalter at oracle.com> > Cc: core-libs-dev at openjdk.java.net > Subject: Re: RFR JDK-8209786: gcc 7.3 compiler errors on zLinux > > Hi Brian, > Thanks for taking a look at this, I have just done a rebuild with a new > patch with appropriate gcc disable warnings for these libraries: > http://cr.openjdk.java.net/~aleonard/8209786/webrev.01/ <http://cr.openjdk.java.net/~aleonard/8209786/webrev.01/> > This works fine, so if you think this is a more favourable approach for > these libraries? i'd like to get this merged please. > Thanks > Andrew > > Andrew Leonard > Java Runtimes Development > IBM Hursley > IBM United Kingdom Ltd > Phone internal: 245913, external: 01962 815913 > internet email: andrewmleonard at uk.ibm.com > > > > > From: Brian Burkhalter <brian.burkhalter at oracle.com> > To: Andrew Leonard <andrewmleonard at uk.ibm.com> > Cc: core-libs-dev at openjdk.java.net > Date: 28/08/2018 15:52 > Subject: Re: RFR JDK-8209786: gcc 7.3 compiler errors on zLinux > > > > Hi Andrew, > > It was suggested that it would be preferable to dial down the compilation > settings for the fdlibm code rather than make a source code change. Was > this investigated? > > Thanks, > > Brian > > On Aug 28, 2018, at 7:18 AM, Andrew Leonard > <andrewmleonard at uk.ibm.com> > wrote: > > We have discovered issues with gcc 7.3 on zLinux, combined with OpenJDK's > default compiler options has highlighted a couple of native code issues, > with undefined behaviours: > - validating loop test array bounds > - left shifts of negative values > I have created bug https://bugs.openjdk.java.net/browse/JDK-8209786 <https://bugs.openjdk.java.net/browse/JDK-8209786> > and attached the webrev fix here: > http://cr.openjdk.java.net/~aleonard/8209786/webrev.00/ <http://cr.openjdk.java.net/~aleonard/8209786/webrev.00/> > > This has already been discussed and refined on the "s390x-port-dev" > maillist > and as it was pointed out, it should have been posted here... > > I'd like to request a sponsor for this fix please? > > > > > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 > 3AU

Unless stated otherwise above: IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

More information about the core-libs-dev mailing list