RFR: 8322322: Support archived full module graph when -Xbootclasspath/a is used [v2]

Calvin Cheung ccheung at openjdk.org
Fri Dec 22 18:33:59 UTC 2023


> Please review this change for enabling full module graph even when -Xbootclasspath/a is specified. The validation of -Xbootclasspath/a is already being done in `FileMapInfo::validate_boot_class_paths()`. The full module graph will be disabled if there's a mismatch in -Xbootclasspath/a between dump time and runtime.
> 
> The changes in ClassLoaders.java is for setting up the append class path for the boot loader retrieved from the CDS archive. It is required because some existing tests are using the `getResourceAsStream()` api. Those tests would fail without the change.
> 
> Passed tiers 1 - 4 testing.

Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:

  comments from Alan and Ioi

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17178/files
  - new: https://git.openjdk.org/jdk/pull/17178/files/b177c6c0..d06b98ff

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17178&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17178&range=00-01

  Stats: 22 lines in 3 files changed: 18 ins; 2 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/17178.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17178/head:pull/17178

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


More information about the core-libs-dev mailing list