RFR: 8286957: Held monitor count [v2]

David Holmes dholmes at openjdk.org
Tue Jun 28 12:43:47 UTC 2022


On Tue, 28 Jun 2022 09:08:04 GMT, Johan Sjölén <duke at openjdk.org> wrote:

>> Robbin Ehn has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>> 
>>  - Merge branch 'master' into held-mon-count
>>  - 8286957 - PR Baseline
>
> src/hotspot/share/interpreter/zero/bytecodeInterpreter.cpp line 629:
> 
>> 627:         mon->lock()->set_displaced_header(displaced);
>> 628:         bool call_vm = UseHeavyMonitors;
>> 629:         bool inc_mon_count = true;
> 
> Should be inc_mon**t**_count? We're consistent with `dec_mont_count`.

Please do not use `mont` as an abbreviation for monitor - use `mon` or `monitor`. Thanks.

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

PR: https://git.openjdk.org/jdk/pull/8945


More information about the hotspot-dev mailing list