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

Kim Barrett kbarrett at openjdk.java.net
Tue Sep 21 22:27:03 UTC 2021


On Fri, 10 Sep 2021 10:52:53 GMT, Thomas Schatzl <tschatzl 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.

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

Marked as reviewed by kbarrett (Reviewer).

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



More information about the hotspot-gc-dev mailing list