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

Ioi Lam iklam at openjdk.java.net
Mon Feb 15 06:40:59 UTC 2021


On Thu, 11 Feb 2021 23:41:34 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: d9744f65
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d9744f65
Stats:     982 lines in 31 files changed: 346 ins; 481 del; 155 mod

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

Reviewed-by: coleenp, ccheung

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

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


More information about the hotspot-dev mailing list