Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-13 Thread Adam Farley8
-- Summary -- I ask for a committer to go into jdk/jdk and add one word to make/lib/Awt2dLibraries.gmk We need to go to line 495 and add array-bounds into the list of disabled warnings. So this: DISABLED_WARNINGS_gcc := clobbered implicit-fallthrough shift-negative-value, \ becomes this:

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-07 Thread Adam Farley8
Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.cErik Joelsson to: Adam Farley8 01/02/2018 17:06 Cc: build-dev, David Holmes, John Paul Adrian Glaubitz, Magnus Ihse Bursie From: Erik Joelsson To: Adam Farley8 Cc: build-dev , David Holmes , John Paul Adrian Glaubitz , Magn

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread Erik Joelsson
Hello, Oracle still builds JDK 10 and 11 using Xcode 6.3 using the Macosx10.9 sdk. We are warning free there. Using newer versions of Xcode is known to produce warnings. I'm currently in the process of updating the toolchain versions and platforms we use at Oracle so this should get fixed in

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread Alan Snyder
> On Feb 1, 2018, at 9:05 AM, Erik Joelsson wrote: > > Regarding warning chasing. I agree that we it's not feasible to chase down > every warning in every version of GCC, or any other toolchain, but I also > think that for platforms/configurations where people are actively developing > change

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread Erik Joelsson
Am I understanding this correctly that it's really not tied to a gcc version but a cpu architecture, so it's only really affecting s390x? Are you also saying that gcc 7.2.1 is also affected but with a different message? I'm fine with disabling this warning conditional on s390x, no need for spec

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread Adam Farley8
> On 02/01/2018 12:36 PM, Adam Farley8 wrote: >> After doing some experimenting, I've discovered the problem isn't limited to >> SLES, or gcc 4.8.5, or zLinux. >> >> (...) >> Thoughts? > I think the problem is more that if you are trying to silence warnings, > you will end up being busy for a ve

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread John Paul Adrian Glaubitz
On 02/01/2018 12:36 PM, Adam Farley8 wrote: After doing some experimenting, I've discovered the problem isn't limited to SLES, or gcc 4.8.5, or zLinux. (...) Thoughts? I think the problem is more that if you are trying to silence warnings, you will end up being busy for a very long time since

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-02-01 Thread Adam Farley8
Hi All, After doing some experimenting, I've discovered the problem isn't limited to SLES, or gcc 4.8.5, or zLinux. Platforms affected: zLinux, ppcle Linux Platforms not affected: x86 Linux gcc versions affected: 4.8.5, 5.4.0 gcc versions unaffected: 7.2.1 linux variations affected: Ubuntu, S

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-01-25 Thread Erik Joelsson
The claim on the wiki was made by Volker on July 13 2017. It was done on SuSE linux while the reported problem is on zLinux. Maybe they have different vendor specific patches in their respective gcc builds? Is SAP still able to build without --disable-warnings-as-errors? This wiki is supposed t

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-01-24 Thread David Holmes
Hi Magnus, On 25/01/2018 9:55 AM, Magnus Ihse Bursie wrote: I'm not sure I'm able to follow where this discussion is heading, but at the very least I'd like to chime in on the patch below: I don't like how the entire DISABLED_WARNINGS_gcc line is lifted out. A better solution is something lik

Re: [OpenJDK 2D-Dev] RFR: Bug Pending: Build fails to compile jchuff.c

2018-01-24 Thread Magnus Ihse Bursie
I'm not sure I'm able to follow where this discussion is heading, but at the very least I'd like to chime in on the patch below: I don't like how the entire DISABLED_WARNINGS_gcc line is lifted out. A better solution is something like this: ifeq ($(CC_VERSION_NUMBER), 4.8.5)  #Turn off array-