RFR: 8371347: Move the ObjectMonitorTable to a separate new file

David Holmes dholmes at openjdk.org
Sun Nov 30 21:40:47 UTC 2025


On Sat, 29 Nov 2025 11:21:52 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.

Seems fine. I did not verify line by line :) 

Thanks

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

Marked as reviewed by dholmes (Reviewer).

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


More information about the hotspot-runtime-dev mailing list