Integrated: 8306733: Remove template parameter of G1DetermineCompactionQueueClosure::free_pinned_region
Thomas Schatzl
tschatzl at openjdk.org
Tue Apr 25 10:43:21 UTC 2023
On Mon, 24 Apr 2023 10:39:30 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
> Hi all,
>
> after [JDK-8298048](https://bugs.openjdk.org/browse/JDK-8298048) G1DetermineCompactionQueueClosure::free_pinned_region() may only be called with humongous regions, so the template parameter is unnecessary. This PR removes it.
> I kept the method itself (not inlined) because I believe its name acts as a good documentation what is done here at the place it is called, even if it's only 3 LOC.
>
> Testing: local compilation, gha
>
> Thanks,
> Thomas
This pull request has now been integrated.
Changeset: a4a53858
Author: Thomas Schatzl <tschatzl at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/a4a5385831b58e66fe3f34cef618643f9be68c9e
Stats: 10 lines in 2 files changed: 0 ins; 6 del; 4 mod
8306733: Remove template parameter of G1DetermineCompactionQueueClosure::free_pinned_region
Reviewed-by: ayang, iwalulya
-------------
PR: https://git.openjdk.org/jdk/pull/13610
More information about the hotspot-gc-dev
mailing list