Integrated: 8329469: Generational ZGC: Move the methods forwarding_[index|find|insert] from zRelocate.cpp to ZForwarding

Guoxiong Li gli at openjdk.org
Thu Apr 11 15:04:48 UTC 2024


On Tue, 2 Apr 2024 09:59:26 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> This patch moves the methods `zRelocate.cpp/forwarding_[index|find|insert]`  from the file `zRelocate.cpp` to the class `ZForwarding`.  The first parameter of the methods `zRelocate.cpp/forwarding_[index|find|insert]` is always `ZForwarding*`. These methods should be placed in the class `ZForwarding` so that the first parameter `ZForwarding*` can be added by the compiler and then it can improve the encapsulation.
> 
> But the non-generational ZGC can't make such a move because the method signatures may be duplicate.
> 
> The test `make test-tier1_gc` passed locally. Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxiong

This pull request has now been integrated.

Changeset: 0656f080
Author:    Guoxiong Li <gli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/0656f0809208160f83a7dd1ae91d9f09b582ce35
Stats:     86 lines in 3 files changed: 40 ins; 34 del; 12 mod

8329469: Generational ZGC: Move the methods forwarding_[index|find|insert] from zRelocate.cpp to ZForwarding

Reviewed-by: stefank, eosterlund

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

PR: https://git.openjdk.org/jdk/pull/18579


More information about the hotspot-gc-dev mailing list