RFR: 8367982: Unify ObjectSynchronizer and LightweightSynchronizer [v2]

Coleen Phillimore coleenp at openjdk.org
Fri Oct 31 12:45:09 UTC 2025


On Mon, 27 Oct 2025 00:29:38 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Fredrik Bredberg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update after review
>
> src/hotspot/share/runtime/synchronizer.cpp line 655:
> 
>> 653:     // installed in the object header.
>> 654:     return install_hash_code(current, obj);
>> 655:   }
> 
> This merging of the OMT and non-OMT `FastHashCode` does not seem related to the `lightweight` renaming. I found it very hard to convince myself of the equivalences here so would prefer to see this in a separate PR.

This was refactored to share with lightweight synchronizer but now doesn't need to be factored out.  The same code is below.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27915#discussion_r2481294207


More information about the hotspot-dev mailing list