RFR: JDK-8164120: The minimal VM should be stripped using --strip-unneeded

[Erik Joelsson]

That was intended but I discovered that the whole thing was wrong. The 
default strip flag we use on Linux "-g" and the new flag we set for 
minimal "--strip-unneeded" are the mutually exclusive, so whatever ends 
up last on the command line wins. Since the new flag is an override, we 
should treat it as such in the makefile logic as well.

Also, the proposed change would have cleared the STRIPFLAGS parameter 
for all other platforms and configurations, because the STRIPFLAGS 
parameter to SetupNativeCompilation accepts an empty value as an 
override. Because of this, we need to give JVM_STRIPFLAGS a default value.

New webrev: http://cr.openjdk.java.net/~erikj/8164120/webrev.hotspot.02/

Properly verified that the minimal JVM is now smaller and that there is 
no difference when building other JVMs.

/Erik


On 2016-10-04 09:32, Magnus Ihse Bursie wrote:
> On 2016-10-03 15:52, Erik Joelsson wrote:
>> Hello,
>>
>> Please review this small change, which adds --strip-unneeded when 
>> stripping the minimal VM on Linux, which reduces size on x86 builds 
>> with around 1.2MB.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8164120
>>
>> Webrev: http://cr.openjdk.java.net/~erikj/8164120/webrev.hotspot.01/
>>
>> /Erik
>>
> Looks good to me. Just a style question: any particular reason you're 
> not using
>
> JVM_STRIPFLAGS += --strip-unneeded
>
> ?
>
> /Magnus
>