On Mon, 29 Mar 2021 17:23:11 GMT, Richard Reingruber <[email protected]> wrote:
>> Robbin Ehn has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains two commits:
>>
>> - Merge branch 'master' into SuspendInHandshake
>> - 8257831: Suspend with handshake (review baseline)
>
> src/hotspot/share/runtime/handshake.cpp line 492:
>
>> 490: }
>> 491: } else {
>> 492: return true;
>
> `true` as return value means that the caller does _not_ need to check again
> for a safepoint, correct? Maybe this could be added as comment to the method
> declaration?
Yes, added comment!
-------------
PR: https://git.openjdk.java.net/jdk/pull/3191
