RFR: 8354083: Support --add-reads with -XX:+AOTClassLinking [v2]
Calvin Cheung
ccheung at openjdk.org
Thu May 15 00:16:32 UTC 2025
> This fix adds the `--add-reads` support for CDS and AOTClassLinking.
> Before the fix, if the `--add-reads` is specified during CDS archive dumping, the user will see the following log if `-Xlog:cds=info` is enabled:
> `[0.000s][info][cds] optimized module handling: disabled due to incompatible property: jdk.module.addreads=com.norequires=org.astro`
> During runtime, the archived full module graph will be disabled:
> `[0.021s][info][cds ] full module graph: disabled`
>
> With the fix, the optimized module handling won't be disabled during dump time and the full module graph will be enabled during runtime provided the same --add-reads option is specified during dump time and runtime.
>
> Testing: tiers 1 - 4.
Calvin Cheung has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
- @iklam comment
- Merge branch 'master' into 8354083-add-reads
- fix trailing whitespace
- 8354083: Support --add-reads with -XX:+AOTClassLinking
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25109/files
- new: https://git.openjdk.org/jdk/pull/25109/files/562fc591..327dc2d5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25109&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25109&range=00-01
Stats: 18384 lines in 682 files changed: 11198 ins; 4135 del; 3051 mod
Patch: https://git.openjdk.org/jdk/pull/25109.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25109/head:pull/25109
PR: https://git.openjdk.org/jdk/pull/25109
More information about the core-libs-dev
mailing list