On Thu, 3 Sep 2026 11:29:46 GMT, Fredrik Bredberg <[email protected]> wrote:

>> Since [JDK-8389325](https://bugs.openjdk.org/browse/JDK-8389325) 
>> `ObjectMonitor::_metadata` only contains a copy of the object’s hash code. 
>> But:
>> 
>> 1. A `ObjectMonitorTable` lookup get the hash directly from the object’s 
>> mark word, and ...
>> 2. ... no synchronization code uses the copy stored in the monitor.
>> 
>> Therefore this PR removes the redundant `_metadata` field and its related 
>> metadata/hash accessors, diagnostics, and serviceability support.
>> 
>> Passes tier1-5 tests successfully on supported platforms.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Fredrik Bredberg 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 four additional 
> commits since the last revision:
> 
>  - For David and Axel
>  - Merge branch 'master' into 8390970_remove_object_monitor_metadata
>  - Updated after review
>  - 8390970: Remove ObjectMonitor::_metadata

Re-approved but I specifically did not ask for that change, so don't blame it 
on me. :)

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32573#pullrequestreview-5108031091

Reply via email to