RFR: 8267846: [lworld] JIT support for the L/Q model (step 1) [v2]

Frederic Parain fparain at openjdk.java.net
Mon May 31 14:13:30 UTC 2021

On Mon, 31 May 2021 14:08:51 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> After https://github.com/openjdk/valhalla/pull/409 and https://github.com/openjdk/valhalla/pull/414 have been integrated, we need to fix the JIT code as well. This mostly means re-applying the old changes from https://github.com/openjdk/valhalla/commit/22dd0da1 and https://github.com/openjdk/valhalla/commit/e895128f and adjust them to the current implementation.
>> All compiler tests now pass except for `TestNullableInlineTypes`, `TestNullableArrays` and `TestIntrinsics` which need reflection support. The corresponding C2 intrinsics are still broken. I'll fix them with [JDK-8267932](https://bugs.openjdk.java.net/browse/JDK-8267932) once reflection support is available.
>> Thanks,
>> Tobias
> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
>   constantTag::internal_name() fix

I cannot review changes in C2 or CI typeFlow, but changes other changes in CI and C1 look good to me.



Marked as reviewed by fparain (Committer).

PR: https://git.openjdk.java.net/valhalla/pull/431

More information about the valhalla-dev mailing list