[lworld] Integrated: 8353202: [lworld] TestArrays.java crashes with -XX:+UseNullableValueFlattening

[Tobias Hartmann]

On Fri, 4 Apr 2025 08:45:39 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> Nullable, atomic, flat arrays and fields of an empty value class should be treated as atomic by C2. Currently, C2 treats them as naturally atomic because they only have one field (the null marker).
> 
> The changes in `src/hotspot/share/c1/c1_LIRGenerator.cpp` are unrelated to this and serve as a temporary workaround for [JDK-8353851](https://bugs.openjdk.org/browse/JDK-8353851) which I found with my new tests.
> 
> Best regards,
> Tobias

This pull request has now been integrated.

Changeset: b5395e82
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/b5395e82b1cce79d3139917b6d151f99ca90ee91
Stats:     72 lines in 6 files changed: 61 ins; 1 del; 10 mod

8353202: [lworld] TestArrays.java crashes with -XX:+UseNullableValueFlattening

-------------

PR: https://git.openjdk.org/valhalla/pull/1418