Integrated: 8315637: JDK-8314249 broke libgraal
Ioi Lam
iklam at openjdk.org
Thu Sep 7 18:32:52 UTC 2023
On Tue, 5 Sep 2023 23:20:17 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> Please review this change in JVMCI. The actual functional change is essentially the same as the patch provided by @dougxc in [JDK-8315637](https://bugs.openjdk.org/browse/JDK-8315637) -- convert the `rawIndex` to `cpci` only if the bytecode is INVOKE{VIRTUAL,SPECIAL,STATIC,INTERFACE}.
>
> The rest of the changes is to rename the parameters from `rawIndex` to `which`, so we know the correct type of index is passed. I also added a test case.
>
> (This code should be much simpler after [JDK-8301993](https://bugs.openjdk.org/browse/JDK-8301993) is complete: we won't have `cpci` anymore and all `which` can be replaced with `rawInex`).
This pull request has now been integrated.
Changeset: 0c865a75
Author: Ioi Lam <iklam at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/0c865a75e658617d40dfa9eb8cf44ccdcea928d9
Stats: 160 lines in 3 files changed: 110 ins; 25 del; 25 mod
8315637: JDK-8314249 broke libgraal
Reviewed-by: dnsimon, matsaave
-------------
PR: https://git.openjdk.org/jdk/pull/15575
More information about the hotspot-compiler-dev
mailing list