RFR: 8371347: Move the ObjectMonitorTable to a separate new file [v2]

Coleen Phillimore coleenp at openjdk.org
Thu Dec 11 19:26:20 UTC 2025


On Thu, 11 Dec 2025 15:17:27 GMT, Fredrik Bredberg <fbredberg at openjdk.org> wrote:

>> The request to move the `ObjectMonitorTable` to a separate new file came from this review comment: https://github.com/openjdk/jdk/pull/27915#discussion_r2459848893
>> 
>> By moving the `ObjectMonitorTable` implementation out of `runtime/synchronizer.cpp` into `runtime/objectMonitorTable.[ch]pp` will make a future transition from the current `ObjectMonitorTable` implementation to a new (and better) implementation appear more clean.
>> 
>> Passes tier1-7 without any problems that can be traced to this PR.
>
> 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:
> 
>  - Merge branch 'master' into 8371347_object_monitor_table_in_separate_file
>  - Update after review
>  - Merge branch 'master' into 8371347_object_monitor_table_in_separate_file
>  - 8371347: Move the ObjectMonitorTable to a separate new file

Thank you!  It looks great!

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28565#pullrequestreview-3568967618


More information about the hotspot-runtime-dev mailing list