> 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:

  simplify some code in modules.cpp

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16016/files
  - new: https://git.openjdk.org/jdk/pull/16016/files/ad4a58d8..471f6414

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

  Stats: 16 lines in 1 file changed: 5 ins; 2 del; 9 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

Reply via email to