On Tue, 13 Aug 2024 14:56:32 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with two 
>> additional commits since the last revision:
>> 
>>  - Remove the last OMWorld references
>>  - Rename omworldtable_work to object_monitor_table_work
>
> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 172:
> 
>> 170:   static void create() {
>> 171:     _table = new ConcurrentTable(initial_log_size(),
>> 172:                                 max_log_size(),
> 
> nit, can you line up these parameters?

Or put them all on L171 if that doesn't make it too long...

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

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

Reply via email to