> 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 incrementally with one 
additional commit since the last revision:

  Un-problem listed test

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1757/files
  - new: https://git.openjdk.org/valhalla/pull/1757/files/12219718..806761aa

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1757&range=03
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1757&range=02-03

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

Reply via email to