Re: [8u] Request for approval for 8172053: (ppc64) Downport of 8170153 breaks build on linux/ppc64 (big endian)

2016-12-28 Thread Erik Joelsson
Change looks ok. /Erik On 2016-12-27 23:30, David Holmes wrote: Hi Volker, As this is a build change I have added build-dev. The change looks fine to me. Aside: More generally it would be nice if there was a simple way to record minimum compiler versions necessary for a given flag/option.

Re: [8u] Request for approval for 8172053: (ppc64) Downport of 8170153 breaks build on linux/ppc64 (big endian)

2016-12-28 Thread Volker Simonis
On Tue, Dec 27, 2016 at 11:30 PM, David Holmes wrote: > Hi Volker, > > As this is a build change I have added build-dev. > > The change looks fine to me. > Thanks for reviewing! > Aside: More generally it would be nice if there was a simple way to record > minimum compiler versions necessary for

RFR(XXS) : 8172094 : 8171433 changes in generated-configure should be restored

2016-12-28 Thread Igor Ignatyev
Hi all, could you please review this tiny fix which restores 8171433 changes in generated-configure file? the changes were lost during latest hs<->dev merge. I’ve regenerated them by runing common/autoconf/autogen.sh. JBS: https://bugs.openjdk.java.net/browse/JDK-8172094 webrev: http://cr.open

Re: RFR(XXS) : 8172094 : 8171433 changes in generated-configure should be restored

2016-12-28 Thread Erik Joelsson
Looks good. /Erik On 2016-12-28 13:27, Igor Ignatyev wrote: Hi all, could you please review this tiny fix which restores 8171433 changes in generated-configure file? the changes were lost during latest hs<->dev merge. I’ve regenerated them by runing common/autoconf/autogen.sh. JBS: https:

Re: RFR(XXS) : 8172094 : 8171433 changes in generated-configure should be restored

2016-12-28 Thread Kirill Zhaldybin
Igor, Looks good to me. Regards, Kirill On 28.12.2016 15:27, Igor Ignatyev wrote: Hi all, could you please review this tiny fix which restores 8171433 changes in generated-configure file? the changes were lost during latest hs<->dev merge. I’ve regenerated them by runing common/autoconf/au