[lworld] Integrated: 8377792: [lworld] compiler/ciReplay/TestLambdas.java fails
Christian Hagedorn
chagedorn at openjdk.org
Fri Feb 13 07:30:06 UTC 2026
On Thu, 12 Feb 2026 13:30:22 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
> When adding compiler replay, I've tried to only emit the additional information value class array properties at replay dump time when we are actually dealing with value classes (i.e. `Arguments::is_valhalla_enabled()` is true). However, this sets an unwanted restriction: When the replay file was created with `--enable-preview` but replay compilation runs without `--enable-preview` (or vice versa), we run into problems. [JDK-8377657](https://bugs.openjdk.org/browse/JDK-8377657) tried to fix this but created new problems. I therefore suggest to just unconditionally emit the array properties, even though they would currently not be used in mainline.
>
> The `ciReplay` tests all passed with and without `--enable-preview` locally.
>
> Thanks,
> Christian
This pull request has now been integrated.
Changeset: 42197b2d
Author: Christian Hagedorn <chagedorn at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/42197b2d459cb689180ba8a2dab21fa773ce93dd
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
8377792: [lworld] compiler/ciReplay/TestLambdas.java fails
Reviewed-by: thartmann
-------------
PR: https://git.openjdk.org/valhalla/pull/2088
More information about the valhalla-dev
mailing list