RFR: 8320147: Remove DumpSharedSpaces [v2]

Ioi Lam iklam at openjdk.org
Sun Nov 19 22:52:02 UTC 2023


> One more PR for cleanup with cdsConfig.hpp:
> 
> Replace the global variable `DumpSharedSpaces` with `CDSConfig::is_dumping_static_archive()`.
> 
> Note: some mis-uses of `DumpSharedSpaces` need to be replaced with `CDSConfig::is_dumping_heap()` or `CDSConfig::is_dumping_full_module_graph()`

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 three additional commits since the last revision:

 - Merge branch 'master' into 8320147-remove-DumpSharedSpaces
 - @calvinccheung comments - fixed copyright
 - 8320147: Remove DumpSharedSpaces

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16700/files
  - new: https://git.openjdk.org/jdk/pull/16700/files/613b5de2..328972de

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

  Stats: 11144 lines in 236 files changed: 4586 ins; 4441 del; 2117 mod
  Patch: https://git.openjdk.org/jdk/pull/16700.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16700/head:pull/16700

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


More information about the hotspot-dev mailing list