RFR: 8261608: Move common CDS archive building code to archiveBuilder.cpp

Ioi Lam iklam at openjdk.java.net
Thu Feb 11 23:51:48 UTC 2021


This is a follow-up to https://git.openjdk.java.net/jdk/pull/2296:

- Move common code for writing the CDS archive from metaspaceShared.cpp to archiveBuilder.cpp

- Data structures related to dumping were haphazardly organized in several classes (e.g., `DumpRegions`). We needed various APIs to access them across classes. These should be consolidated in archiveBuilder.cpp and the API should be cleaned up

- Detailed stats (`DumpAllocStats::print_stats`) were available only for static dump. Refactor the code so they are also printed for dynamic dump

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

Commit messages:
 - 8261608: Move common CDS archive building code to archiveBuilder.cpp

Changes: https://git.openjdk.java.net/jdk/pull/2536/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2536&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8261608
  Stats: 943 lines in 31 files changed: 327 ins; 464 del; 152 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2536.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2536/head:pull/2536

PR: https://git.openjdk.java.net/jdk/pull/2536


More information about the hotspot-dev mailing list