RFR: 8273590: Move helper classes in G1 post evacuation sub tasks to cpp files

Thomas Schatzl tschatzl at openjdk.java.net
Wed Sep 22 08:07:04 UTC 2021


On Tue, 21 Sep 2021 22:24:07 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Hi all,
>> 
>>   can I have reviews for this straightforward refactoring, moving helper classes that are not required to .cpp files. The nice side effect of this is that it removes around 100 LOC. No changes to code, except for removal of one unused local variable in
>> ```RemoveSelfForwardPtrsTask::~RemoveSelfForwardPtrsTask`.
>> 
>> Testing: compilation, gha, local jtreg/gc/g1
>> 
>> Thanks,
>>   Thomas
>
> Looks good.

Thanks @kimbarrett @kstefanj for your reviews

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

PR: https://git.openjdk.java.net/jdk/pull/5461



More information about the hotspot-gc-dev mailing list