On Wed, 20 Sep 2023 17:36:06 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

> Please review this trivial change to remove the UseHeavyMonitors develop 
> option, in favor of the now non-experimental LockingMode=LM_MONITOR (0) 
> option.  Tested with tier1 locally.

Changes requested by dcubed (Reviewer).

src/hotspot/share/runtime/globals.hpp line 1064:

> 1062:   develop(bool, VerifyHeavyMonitors, false,                             
>     \
> 1063:           "Checks that no stack locking happens when using "            
>     \
> 1064:           "-XX:LockingMode=LM_MONITOR")                                 
>     \

s/LM_MONITOR/0/

I don't think you can specify the `LM_MONITOR` value on the actual cmd line.

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

PR Review: https://git.openjdk.org/jdk/pull/15845#pullrequestreview-1636204262
PR Review Comment: https://git.openjdk.org/jdk/pull/15845#discussion_r1331988698

Reply via email to