RFR(XS): 8211837: Creation of the default CDS Archive should depend on ENABLE_CDS
martin.doerr at sap.com
Mon Oct 8 09:10:40 UTC 2018
looks good. Thanks for fixing.
Of course, it would be great if this could be used to fix minimal/zero build, too.
From: hotspot-runtime-dev <hotspot-runtime-dev-bounces at openjdk.java.net> On Behalf Of Volker Simonis
Sent: Montag, 8. Oktober 2018 10:19
To: build-dev <build-dev at openjdk.java.net>; hotspot-runtime-dev at openjdk.java.net runtime <hotspot-runtime-dev at openjdk.java.net>
Subject: RFR(XS): 8211837: Creation of the default CDS Archive should depend on ENABLE_CDS
can I please have a review for the following trivial build fix:
It makes no sense to try to create a default CDS archive on VMs which
don't support CDS at all. We already have '--enable_cds' which
defaults to 'true' on all platforms except AIX.
The check for '--enable_cds_archive' should use the result of
'--enable_cds' (which is saved in "ENABLE_CDS") and only enable
default archive creation if CDS is enabled.
Failure to do so currently breaks the AIX build (after JDK-)8202951 was pushed).
Thank you and best regards,
More information about the hotspot-runtime-dev