On Tue, 16 Dec 2025 09:13:12 GMT, Paul Hübner <[email protected]> wrote:
>> 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.
I think it makes sense to push this and similar changes into jdk/jdk to
minimize diff for valhalla.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1793#discussion_r2624032539
