RFR: 8231672: Simplify the reference processing parallelization framework [v5]
Leo Korinth
lkorinth at openjdk.java.net
Tue Apr 27 12:54:47 UTC 2021
On Sat, 24 Apr 2021 09:45:28 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
>> Leo Korinth has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>>
>> - make RefProcTask a StackObj
>> - Merge branch 'master' into _8231672
>> - using proxies instead of ClosureContexts
>> - remove maybe prefix, change in one place to nullable
>> - Merge branch 'master' into _8231672
>> - Changes after first review
>> - 8231672: Simplify the reference processing parallelization framework
>
> src/hotspot/share/gc/shared/referenceProcessor.cpp line 224:
>
>> 222:
>> 223: {
>> 224: RefProcTotalPhaseTimesTracker tt(RefPhase1, &phase_times, this);
>
> `RefProcTotalPhaseTimesTracker` doesn't really use `this`, right?
Nice catch! Pre-existing, but I will clean it up.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2782
More information about the hotspot-gc-dev
mailing list