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).

Shenandoah parts look OK.

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

> 83: //  - klass - klass identifier used when UseCompactObjectHeaders == true
> 84: 
> 85: class ObjectMonitor;

One of our analysis tools points out that `basicLock.hpp` used this forward 
declaration for its `ObjectMonitor* volatile _monitor`. Put the forward 
declaration there?

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/32471#pullrequestreview-5006504353
PR Review Comment: https://git.openjdk.org/jdk/pull/32471#discussion_r3842317811

Reply via email to