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 hotspot-compiler-dev
mailing list