[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