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

Ioi Lam iklam at openjdk.java.net
Sat Feb 13 05:04:58 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

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  review comments by @calvinccheung

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2536/files
  - new: https://git.openjdk.java.net/jdk/pull/2536/files/9582e40f..54e7185f

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2536&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2536&range=01-02

  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 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