Re: RFR: JDK-8171310: Gtest libjvm.so is always stripped
On 2016-12-15 21:29, Thomas Stüfe wrote: Short question, how do you generate and apply a webrev patch spanning multiple forest branches? If you have the trees extension, there is a parameter to webrev, "-f" (for forest, historical reasons..) that does this. /Erik
Re: RFR: JDK-8171310: Gtest libjvm.so is always stripped
Hi Erik, Seems reasonable/logical. :) Thanks, David On 16/12/2016 1:01 AM, Erik Joelsson wrote: Hello, Please review this patch, which fixes stripping behavior in the build. In JDK-8150736 I reworked the general handling of debug symbols. My intention then was to always leave debug symbols in gtest libjvm.so as well as all other test binaries. It seems I forgot to set this for gtest libjvm.so however. While investigating this I also discovered that setting --with-native-debug-symbols=internal doesn't work. Most binaries are stripped anyway. This is a bug in NativeCompilation.gmk. My fix changes the default behavior for stripping to only happen if debug symbols are also copied. Since the global defaults are now sane and universal, I also removed all explicit STRIP_SYMBOLS:=true settings from specific native libraries. It doesn't make sense to strip them if the user asked for internal debug symbols and in all other cases they will be stripped anyway. Bug: https://bugs.openjdk.java.net/browse/JDK-8171310 Webrev: http://cr.openjdk.java.net/~erikj/8171310/webrev.01/ /Erik
Re: RFR: JDK-8171310: Gtest libjvm.so is always stripped
Erik, On Thu, Dec 15, 2016 at 4:01 PM, Erik Joelsson wrote: > Hello, > > Please review this patch, which fixes stripping behavior in the build. In > JDK-8150736 I reworked the general handling of debug symbols. My intention > then was to always leave debug symbols in gtest libjvm.so as well as all > other test binaries. It seems I forgot to set this for gtest libjvm.so > however. > > While investigating this I also discovered that setting > --with-native-debug-symbols=internal doesn't work. Most binaries are > stripped anyway. This is a bug in NativeCompilation.gmk. My fix changes the > default behavior for stripping to only happen if debug symbols are also > copied. > > Since the global defaults are now sane and universal, I also removed all > explicit STRIP_SYMBOLS:=true settings from specific native libraries. It > doesn't make sense to strip them if the user asked for internal debug > symbols and in all other cases they will be stripped anyway. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8171310 > > Webrev: http://cr.openjdk.java.net/~erikj/8171310/webrev.01/ > > /Erik > > Thank you for fixing this. Works, I am now able to debug the gtest libjvm.so. Short question, how do you generate and apply a webrev patch spanning multiple forest branches? Thanks, Thomas
Re: RFR: JDK-8171310: Gtest libjvm.so is always stripped
Erik: Please review this patch, which fixes stripping behavior in the build. In JDK-8150736 I reworked the general handling of debug symbols. My intention then was to always leave debug symbols in gtest libjvm.so as well as all other test binaries. It seems I forgot to set this for gtest libjvm.so however. While investigating this I also discovered that setting --with-native-debug-symbols=internal doesn't work. Most binaries are stripped anyway. This is a bug in NativeCompilation.gmk. My fix changes the default behavior for stripping to only happen if debug symbols are also copied. Since the global defaults are now sane and universal, I also removed all explicit STRIP_SYMBOLS:=true settings from specific native libraries. It doesn't make sense to strip them if the user asked for internal debug symbols and in all other cases they will be stripped anyway. Bug: https://bugs.openjdk.java.net/browse/JDK-8171310 Webrev: http://cr.openjdk.java.net/~erikj/8171310/webrev.01/ Looks good. Tim