[lworld] Integrated: 8288098: [lworld] C2 fails to scalarize value class arguments
Tobias Hartmann
thartmann at openjdk.java.net
Thu Jun 9 13:08:56 UTC 2022
On Thu, 9 Jun 2022 11:49:35 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> C2 fails to scalarize a value class argument because its class is not listed for preloading (because the argument class is equal to the method holder class). We should not rely on the preload attribute here but simply check if the argument class is a loaded value class. Proper mismatch handling will be added by [JDK-8284443](https://bugs.openjdk.org/browse/JDK-8284443).
>
> I also added a regression test and fixed unrelated `TestNullableInlineTypes::test97` which failed due to recent changes.
>
> Thanks,
> Tobias
This pull request has now been integrated.
Changeset: 328d1ea2
Author: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.java.net/valhalla/commit/328d1ea273241c67654174227c1145a2256609bf
Stats: 29 lines in 6 files changed: 11 ins; 12 del; 6 mod
8288098: [lworld] C2 fails to scalarize value class arguments
-------------
PR: https://git.openjdk.java.net/valhalla/pull/709
More information about the valhalla-dev
mailing list