RFR: 8267846: [lworld] JIT support for the L/Q model (step 1) [v2]
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.
> 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).
More information about the valhalla-dev