Integrated: 8267112: JVMCI compiler modules should be kept upgradable

Vladimir Kozlov kvn at openjdk.java.net
Mon May 17 20:13:43 UTC 2021


On Thu, 13 May 2021 16:37:38 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> [JDK-8264806](https://bugs.openjdk.java.net/browse/JDK-8264806) changes removed sources and also removed JVMCI compiler from list of upgradable modules. JVMCI compiler modules should be upgradable in JDK to work with GraalVM. 
> 
> Make these modules upgradable again and empty by leaving only reference to JVMCI (jdk.internal.vm.ci) module. It does not restore sources - only `module-info.java` files are kept.
> 
> Note, we continue discussion about [JDK-8265091](https://bugs.openjdk.java.net/browse/JDK-8265091): "Use Module API to export JVMCI packages at runtime" to see if we can remove these `module-info.java` files.
> 
> Changes were proposed by @dougxc after testing [JDK-8264806](https://bugs.openjdk.java.net/browse/JDK-8264806) changes with GraalVM.
> I restored related code in some tests for them to pass.
> 
> Testing: full tier1-tier3.

This pull request has now been integrated.

Changeset: 2effdd1b
Author:    Vladimir Kozlov <kvn at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2effdd1b6799a15a766b2b2a6cba4806d92122f3
Stats:     83 lines in 9 files changed: 34 ins; 42 del; 7 mod

8267112: JVMCI compiler modules should be kept upgradable

Reviewed-by: mchung, erikj, dnsimon

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

PR: https://git.openjdk.java.net/jdk/pull/4014



More information about the build-dev mailing list