[lworld] RFR: 8370904: [lworld] Enable CDS heap dumping [v2]

Matias Saavedra Silva matsaave at openjdk.org
Tue Dec 16 19:15:10 UTC 2025


> CDS heap dumping was manually disabled in lworld due to the method in which the migrated value classes were added to the JDK. Migrated value classes were added through module patching which is incompatible with CDS heap dumping, but now since they are added through jimage, we can re-enable CDS heap dumping and related features that rely on it.
> 
> AOT adds some complexity to this as it loads and links classes in VM startup, so alternate code paths need to be considered. Signature calling conventions are stored in adapters which are archived in the AOT code cache, but they aren't usable at runtime, so they need to be regenerated even if we don't regenerate the adapters.

Matias Saavedra Silva has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:

 - Merge branch 'lworld' into enable_cds_heap
 - Reverted preview patching flag
 - Fred comment
 - Removed test from group
 - Merge branch 'lworld' into enable_cds_heap
 - Merge branch 'lworld' into enable_cds_heap
 - Properly rewrite fast_vputfield and fast_vgetfield
 - Merge branch 'lworld' into enable_cds_heap
 - Merge branch 'lworld' into enable_cds_heap
 - Enable CDS heap dumping

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

Changes: https://git.openjdk.org/valhalla/pull/1757/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1757&range=01
  Stats: 69 lines in 8 files changed: 37 ins; 28 del; 4 mod
  Patch: https://git.openjdk.org/valhalla/pull/1757.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1757/head:pull/1757

PR: https://git.openjdk.org/valhalla/pull/1757


More information about the valhalla-dev mailing list