On Thu, 20 Aug 2026 14:13:27 GMT, Fredrik Bredberg <[email protected]> 
wrote:

> This PR is a follow-up to 
> [JDK-8389325](https://bugs.openjdk.org/browse/JDK-8389325) ("Remove the 
> UseObjectMonitorTable flag and related code"). It:
> - Removes obsolete mark-word and monitor-pointer support.
> - Removes displaced-mark related code in garbage collectors (G1, Parallel GC 
> and Shenandoah).
> - Renames UseObjectMonitorTableTest to ObjectMonitorTableTest.
> - Addresses all the leftover review comments from 
> [JDK-8389325](https://bugs.openjdk.org/browse/JDK-8389325).
> 
> 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).

Looks good to me, thanks for the cleanup.

src/hotspot/share/oops/markWord.hpp line 275:

> 273:     return has_monitor_pointer();
> 274:   }
> 275:   markWord displaced_mark_helper() const;

I see the SA agent has the equivalent methods 
`hasDisplacedMarkHelper/displacedMarkHelper`.

src/hotspot/share/runtime/objectMonitor.hpp line 232:

> 230:   //
> 231:   #define OM_OFFSET_NO_MONITOR_VALUE_TAG(f) \
> 232:     ((in_bytes(ObjectMonitor::f ## _offset())) - 
> checked_cast<int>(markWord::monitor_value))

This macro can also be removed.

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

Marked as reviewed by pchilanomate (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32471#pullrequestreview-4994531261
PR Review Comment: https://git.openjdk.org/jdk/pull/32471#discussion_r3831214227
PR Review Comment: https://git.openjdk.org/jdk/pull/32471#discussion_r3831206223

Reply via email to