RFR: 8231672: Simplify the reference processing parallelization framework [v4]
Leo Korinth
lkorinth at openjdk.java.net
Tue Mar 23 13:40:43 UTC 2021
On Mon, 1 Mar 2021 18:25:44 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Leo Korinth has updated the pull request incrementally with one additional commit since the last revision:
>>
>> remove maybe prefix, change in one place to nullable
>
> src/hotspot/share/gc/shared/referenceProcessor.hpp line 596:
>
>> 594: virtual VoidClosure* complete_gc(uint worker_id) = 0;
>> 595: virtual void prepare_run_task(uint queue_count, ThreadModel tm, bool marks_oops_alive) = 0;
>> 596: uint index(uint id, ThreadModel tm) { return (tm==ThreadModel::Single)?0:id; }
>
> I'd like more spaces around operators and punctuation in that value expression.
The code was updated because Thomas also found it, are you okay with the current code?
-------------
PR: https://git.openjdk.java.net/jdk/pull/2782
More information about the hotspot-gc-dev
mailing list