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