RFR: 8265435: Remove dummy lists in G1CalculatePointersClosure

Stefan Johansson sjohanss at openjdk.java.net
Tue Apr 20 08:37:05 UTC 2021


On Mon, 19 Apr 2021 12:20:21 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

> Remove the dummy list and some general cleanup.

Nice cleanup. Just a small naming suggestion :)

src/hotspot/share/gc/g1/g1FullGCPrepareTask.cpp line 43:

> 41: 
> 42: template<bool is_humongous>
> 43: void G1FullGCPrepareTask::G1CalculatePointersClosure::free_region(HeapRegion* hr) {

Since this is only for "pinned regions", maybe call this `free_pinned_region()`.

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

Marked as reviewed by sjohanss (Reviewer).

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



More information about the hotspot-gc-dev mailing list