RFR: 8318027: Support alternative name to jdk.internal.vm.compiler [v2]

Doug Simon dnsimon at openjdk.org
Fri Oct 20 15:32:55 UTC 2023


> The Graal code base has [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33) its module to `jdk.compiler.graal` as part of preparations for Project Galahad. Due to the way Java modules work, this requires a JDK change. The core of the issue is that the [service](https://github.com/openjdk/jdk/blob/56aa1e8dc8047cbc29d554889c64beb6eca0b8eb/src/jdk.internal.vm.ci/share/classes/module-info.java#L37) by which HotSpot requests a Graal compilation is defined in JVMCI. Since JVMCI is in the boot layer, the service can only be implemented by a provider in the boot layer and the package defining the service must be exported to the provider's defining module. This export currently targets [`jdk.internal.vm.compiler`](https://github.com/openjdk/jdk/blob/56aa1e8dc8047cbc29d554889c64beb6eca0b8eb/src/jdk.internal.vm.ci/share/classes/module-info.java#L28) and so the binding fails for the new Graal module. To address this, this PR reflects the Graal module rena
 ming, including adjusting the qualified export.

Doug Simon has updated the pull request incrementally with one additional commit since the last revision:

  fix copyright dates and @since tags to reflect history

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16189/files
  - new: https://git.openjdk.org/jdk/pull/16189/files/6b729561..cc0474b2

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

  Stats: 4 lines in 2 files changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/16189.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16189/head:pull/16189

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


More information about the build-dev mailing list