RFR: 8316969: Improve CDS module graph support for --module option [v5]

Calvin Cheung ccheung at openjdk.org
Thu Oct 26 20:59:58 UTC 2023


> Please review this changeset for adding support for `--module` (-m) option for CDS.
> Changes in the `ModuleBootstrap.java` are needed so that the `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if the main module is specified. The module name will be stored in the ro region of the CDS archive. During runtime, the archived module name will be compared with the runtime module name. If comparison fails, the archived full module graph won't be used.
> 
> Note: this RFE is a subtask of [JDK-8266329](https://bugs.openjdk.org/browse/JDK-8266329). More subtask(s) will be created to support other options such as `--add-modules`.
> 
> Passed tiers 1 - 4 testing.

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

  skip archiving full module graph is there is an incubator module

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16016/files
  - new: https://git.openjdk.org/jdk/pull/16016/files/6c82fde0..f6b2c3d5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16016&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16016&range=03-04

  Stats: 81 lines in 5 files changed: 59 ins; 1 del; 21 mod
  Patch: https://git.openjdk.org/jdk/pull/16016.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16016/head:pull/16016

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


More information about the core-libs-dev mailing list