[lworld] RFR: 8373575: [lworld] Fix and problem list failing JDK, langtools and HotSpot tier 1 tests under --enable-preview [v5]
Paul Hübner
phubner at openjdk.org
Tue Dec 16 09:15:40 UTC 2025
On Mon, 15 Dec 2025 21:07:07 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> Paul Hübner has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>>
>> - Merge branch 'lworld' into JDK-8373575
>> - Move some tests to Xcomp list.
>> - Add failing test to problemlist.
>> - Add one of the AOT tests.
>> - Problemlist and fix.
>
> test/hotspot/jtreg/compiler/escapeAnalysis/Test8329757.java line 46:
>
>> 44: void q() {
>> 45: int e;
>> 46: synchronized (new Object()) {
>
> What motivated this change?
We cannot synchronize on value classes (primitive boxes are migrated to value classes with preview mode enabled) and we throw an `IdentityException` if we attempt to do so. I've talked with the author of the test and compiler folks and it doesn't change the semantics if we change the mutex.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1793#discussion_r2622436048
More information about the valhalla-dev
mailing list