Integrated: 8320935: Move CDS config initialization code to cdsConfig.cpp

Ioi Lam iklam at openjdk.org
Wed Dec 6 05:28:41 UTC 2023


On Tue, 28 Nov 2023 23:24:53 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> This is a simple clean up that moves the code for initializing the CDS config states from arguments.cpp to cdsConfig.cpp
> 
> I renamed a few functions, but otherwise the code is unchanged.
> 
> - `get_default_shared_archive_path()` -> `default_archive_path()`
> - `GetSharedArchivePath()` -> `static_archive_path()`
> - `GetSharedDynamicArchivePath()` -> `dynamic_archive_path()`
> 
> There's also less `#if INCLUDE_CDS` since the entire cdsConfig.cpp file is compiled only if CDS is enabled.

This pull request has now been integrated.

Changeset: 4c96aac9
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4c96aac9c0aa450b0b6859ded8dfff856222ad58
Stats:     696 lines in 8 files changed: 346 ins; 327 del; 23 mod

8320935: Move CDS config initialization code to cdsConfig.cpp

Reviewed-by: ccheung, matsaave, stuefe

-------------

PR: https://git.openjdk.org/jdk/pull/16868


More information about the hotspot-dev mailing list