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