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