[jdk11u-dev] RFR: 8285395: [JVMCI] Partial backport of JDK-8220623: InstalledCode

[Andrew Dinn]

On Thu, 21 Apr 2022 17:45:16 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

> Please review this 11u-only change in the JVMCI area. Latest Graal master introduced a change which overrides `getAddress()` and `getEntryPoint()` in class `InstalledCode`. This isn't an issue in Graal VM CE since labs openjdk is used there which has newer JVMCI level. The same code is present in JDK 13+. I'm only proposing a partial backport since that's sufficient to unbreak the Mandrel build and is a lot less risky than bringing in JDK-8220623 wholesale. See the bug for details.
> 
> Testing:
>  - [x] jvmci tests
>  - [x] Mandrel build with a base JDK including this patch. Builds now, broke earlier.

This partial backport is the right way to go here and the change looks fine to me.

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

Marked as reviewed by adinn (Reviewer).

PR: https://git.openjdk.java.net/jdk11u-dev/pull/1038