RFR: 8318484: Initial version of cdsConfig.hpp [v2]

Ioi Lam iklam at openjdk.org
Fri Oct 20 23:04:47 UTC 2023


> This is the first step for [JDK-8318483 - Move CDS configuration management into cdsConfig.hpp](https://bugs.openjdk.org/browse/JDK-8318483)
> 
> - Remove `Arguments::is_dumping_archive()` and `Arguments assert_is_dumping_archive()`
> - Add the following new APIs
> 
> 
> class CDSConfig {
>   static bool is_dumping_archive();
>   static bool is_dumping_static_archive();
>   static bool is_dumping_dynamic_archive();
>   static bool is_dumping_heap();
> };
> 
> 
> - Convert some use of `DumpSharedSpaces` and `DynamicDumpSharedSpaces` to these new APIs
> 
> (More APIs will be added in future sub tasks of [JDK-8318483](https://bugs.openjdk.org/browse/JDK-8318483))

Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:

 - Merge branch 'master' into 8318484-initial-version-of-cdsConfig-hpp
 - 8318484: Initial version of cdsConfig.hpp

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16257/files
  - new: https://git.openjdk.org/jdk/pull/16257/files/0a2f78b0..0d729778

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16257&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16257&range=00-01

  Stats: 5386 lines in 163 files changed: 3808 ins; 820 del; 758 mod
  Patch: https://git.openjdk.org/jdk/pull/16257.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16257/head:pull/16257

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


More information about the serviceability-dev mailing list