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