Integrated: 8348349: Refactor CDSConfig::is_dumping_heap()

Ioi Lam iklam at openjdk.org
Tue Feb 4 16:11:20 UTC 2025


On Thu, 23 Jan 2025 02:35:06 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> Please review this small clean up:
> 
> `HeapShared::can_write()` and `CDSConfig::is_dumping_heap()` are both for deciding whether CDS should dump heap objects. I removed the former and consolidated all the logic to the latter.
> 
> I also updated the logging message in case heap objects cannot be dumped.
> 
> I also updated VMProps to clarify what `vmCDSCanWriteArchivedJavaHeap()` means.

This pull request has now been integrated.

Changeset: b985347c
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b985347c2383a7a637ffa9a4a8687f7f7cde1369
Stats:     111 lines in 12 files changed: 52 ins; 36 del; 23 mod

8348349: Refactor CDSConfig::is_dumping_heap()

Reviewed-by: ccheung, matsaave

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

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


More information about the hotspot-dev mailing list