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