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