RFR: JDK-8304820: Statically allocate ObjectSynchronizer mutexes [v2]
Justin King
jcking at openjdk.org
Thu Mar 23 21:48:52 UTC 2023
On Thu, 23 Mar 2023 20:56:36 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> Justin King has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update existing spacing since line is being touched anyway
>>
>> Signed-off-by: Justin King <jcking at google.com>
>
> src/hotspot/share/runtime/synchronizer.cpp line 743:
>
>> 741: // Index into the lock array based on the current object address.
>> 742: static_assert(is_power_of_2(ARRAY_SIZE(gInflationLocks)), "must be");
>> 743: size_t ix = (cast_from_oop<intptr_t>(obj) >> 5) & (ARRAY_SIZE(gInflationLocks)-1);
>
> Since you are touching the line:
> nit: s/-1/ - 1/
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13160#discussion_r1146901711
More information about the hotspot-runtime-dev
mailing list