On Mon, 12 Aug 2024 18:55:41 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Missing DEBUG_ONLY
>
> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 341:
> 
>> 339: };
>> 340: 
>> 341: ObjectMonitorTable* LightweightSynchronizer::_omworld = nullptr;
> 
> I preferred my version where ObjectMonitorTable was AllStatic which gets rid 
> of _omworld-> everwhere, but the internal table is the pointer.  You can 
> remove more omworld names also.

I implemented the AllStatic table, and remove all mentions of OMWorld.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20067#discussion_r1715432994

Reply via email to