RFR: 8367601: Remove held_monitor_count [v2]

Fredrik Bredberg fbredberg at openjdk.org
Wed Oct 1 13:40:00 UTC 2025


On Tue, 30 Sep 2025 15:51:14 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

>> Fredrik Bredberg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update after review
>
> src/hotspot/share/runtime/continuationFreezeThaw.cpp line 1742:
> 
>> 1740:     log_develop_debug(continuations)("PINNED due to critical section");
>> 1741:     verify_continuation(cont.continuation());
>> 1742:     freeze_result res = entry->is_pinned() ? freeze_pinned_cs : freeze_pinned_monitor;
> 
> We can remove this and always return freeze_pinned_cs. We should remove freeze_pinned_monitor (there is a matching definition in Continuation.java).

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27570#discussion_r2394622676


More information about the graal-dev mailing list