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

Thomas Schatzl tschatzl at openjdk.java.net
Wed Sep 22 08:11:02 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

This pull request has now been integrated.

Changeset: d9872ba3
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d9872ba3d6c5afb2ae8e17c38e2222a2a1ab9493
Stats:     423 lines in 2 files changed: 108 ins; 190 del; 125 mod

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

Reviewed-by: sjohanss, kbarrett

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

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



More information about the hotspot-gc-dev mailing list