On Mon, 22 Sep 2025 11:01:42 GMT, Viktor Klang <[email protected]> wrote:
>> Doug Lea has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add @Contended WorkQueue region. Adapt accordingly.
>
> src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 2118:
>
>> 2116: private int tryTrim(WorkQueue w, int activePhase, long deadline) {
>> 2117: long c, nc; int stat, vp, i; WorkQueue[] vs; WorkQueue v;
>> 2118: long waitTime = deadline - System.currentTimeMillis();
>
> Why hoist this line from the branch? 🤔
It makes the no-longer-eligible check a teeny bit more accurate.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26479#discussion_r2368043344
