Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-24 Thread Erik Joelsson
led always for LIB_JVM). Best regards, Matthias From: Baesken, Matthias Sent: Freitag, 17. Januar 2020 12:44 To: Magnus Ihse Bursie ; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: RE: RFR: 8236714: enable link-time section-gc for linux to remove u

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-24 Thread Baesken, Matthias
Thanks for the review ! Erik, are you fine with the latest change too ? Best regards, Matthias > On 2020-01-24 10:27, Baesken, Matthias wrote: > > Hi Erik, thanks for the comments, new webrev : > > > > http://cr.openjdk.java.net/~mbaesken/webrevs/8236714.7/ > Looks good to me. Sorry for not

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-24 Thread Magnus Ihse Bursie
; 'hotspot-...@openjdk.java.net' Subject: RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code * Matthias: Have a look at some recently added option to get an indication of the best practice in adding new options. There are some ways to easily make this inc

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-24 Thread Baesken, Matthias
be set to enable the link time section gc . > >>> Exception is linuxs390x where we already have the feature enabled > (and > >> keep it enabled always for LIB_JVM). > >>> Best regards, Matthias > >>> > >>> > >>> > >>> F

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-23 Thread Erik Joelsson
; 'hotspot-...@openjdk.java.net' Subject: RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code * Matthias: Have a look at some recently added option to get an indication of the best practice in adding new options. There are some ways to easily make this incorrect Hi Magnus, d

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-23 Thread Baesken, Matthias
: Baesken, Matthias > > Sent: Freitag, 17. Januar 2020 12:44 > > To: Magnus Ihse Bursie ; David Holmes > ; 'build-dev@openjdk.java.net' d...@openjdk.java.net>; 'hotspot-...@openjdk.java.net' d...@openjdk.java.net> > > Subject: RE: RFR: 8236714: enable link-time section-

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-22 Thread Baesken, Matthias
Hi Erik, okay I'll check that . I had the impression I would have ordering issues of the m4 files and how they end up in the generated-configure.sh but looks like that’s not the case . Best regards, Matthias > > Hello Matthias, > > You can keep the setting up of all the flags in

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-22 Thread Erik Joelsson
Ihse Bursie ; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code * Matthias: Have a look at some recently added option to get an indication of the best practice in adding

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-22 Thread Baesken, Matthias
(and keep it enabled always for LIB_JVM). Best regards, Matthias From: Baesken, Matthias Sent: Freitag, 17. Januar 2020 12:44 To: Magnus Ihse Bursie ; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: RE: RFR: 8236714: enable link-time section-gc for linux

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-17 Thread Baesken, Matthias
* Matthias: Have a look at some recently added option to get an indication of the best practice in adding new options. There are some ways to easily make this incorrect Hi Magnus, do you have a good/”best practice” example (not that I catch a bad one  ) ? Best regards, Matthias

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-17 Thread Magnus Ihse Bursie
On 2020-01-16 10:30, David Holmes wrote: Hi Matthias, On 16/01/2020 6:10 pm, Baesken, Matthias wrote: Hi David, sure we can introduce a way to switch this on/off. Thanks. There is already something similar for the link-time optimization (flto) , see the feature JvmFeatures.gmk 180 ifeq

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-16 Thread David Holmes
Hi Matthias, On 16/01/2020 6:10 pm, Baesken, Matthias wrote: Hi David, sure we can introduce a way to switch this on/off. Thanks. There is already something similar for the link-time optimization (flto) , see the feature JvmFeatures.gmk 180 ifeq ($(call check-jvm-feature, link-time-opt),

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-16 Thread Baesken, Matthias
Hi David, sure we can introduce a way to switch this on/off. There is already something similar for the link-time optimization (flto) , see the feature JvmFeatures.gmk 180 ifeq ($(call check-jvm-feature, link-time-opt), true) 190 ifeq ($(call check-jvm-feature, link-time-opt), false)

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-15 Thread David Holmes
Hi Matthias, I have reservations about turning this on by default and with no way to control it. This seems like it should be something you have to opt-in to initially while we gain some experience with it and ensure there are no unexpected side-effects. After that it could be enabled by

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-15 Thread Baesken, Matthias
Hi Erik, I did not notice slowdowns in our night makes . Looking at a specific test machine I use (x86_64, build JOBS hardwired set to 12 ) I get around 6 minutes build time with and without the feature . ( but you have to take into account that the link-time section-gc on

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-15 Thread Erik Joelsson
Given the discussion regarding lto on hotspot and the extreme increased build time, have you noticed any difference in build times with this patch? /Erik On 2020-01-15 00:27, Baesken, Matthias wrote: Hi Erik, thanks for the review and for forwarding , you are correct corelibs-dev is

RE: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-15 Thread Baesken, Matthias
Hi Erik, thanks for the review and for forwarding , you are correct corelibs-dev is probably interested in this as well . Best regards, Matthias > (adding core-libs-dev) > > Change looks good to me, but would like input from at least someone in > core-libs. > > /Erik > > On 2020-01-14

Re: RFR: 8236714: enable link-time section-gc for linux to remove unused code

2020-01-14 Thread Erik Joelsson
(adding core-libs-dev) Change looks good to me, but would like input from at least someone in core-libs. /Erik On 2020-01-14 06:07, Baesken, Matthias wrote: Hello, the following change enables the link-time section-gc for linux . gcc and ld support enabling "garbage collection" of unused