RFR: 8338383: Implement JEP 491: Synchronize Virtual Threads without Pinning [v11]
    Patricio Chilano Mateo 
    pchilanomate at openjdk.org
       
    Fri Oct 25 18:50:24 UTC 2024
    
    
  
On Fri, 25 Oct 2024 05:25:58 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Patricio Chilano Mateo has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Add/fix comments for David
>>  - Move condition to new line in nmethod::preserve_callee_argument_oops
>
> src/hotspot/share/runtime/objectMonitor.cpp line 1698:
> 
>> 1696:     // on _WaitSetLock so it's not profitable to reduce the length of the
>> 1697:     // critical section.
>> 1698: 
> 
> Please restore the blank line, else it looks like the comment block pertains to the `wait_reenter_begin`, but it doesn't.
Restored.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1817199027
    
    
More information about the graal-dev
mailing list