Integrated: 8279910: G1: Simplify HeapRegionRemSet::add_reference

Hamlin Li mli at openjdk.java.net
Mon Jan 17 14:36:27 UTC 2022


On Wed, 12 Jan 2022 08:29:29 GMT, Hamlin Li <mli at openjdk.org> wrote:

> HeapRegionRemSet::add_reference checks "state == Untracked" and return if true; it's called at G1RebuildRemSetClosure::do_oop_work and G1ConcurrentRefineOopClosure::do_oop_work respectively.
> 
> The check in HeapRegionRemSet::add_reference could be replaced with an assert, and move the check up to one of the caller G1RebuildRemSetClosure::do_oop_work.

This pull request has now been integrated.

Changeset: 5d52bf99
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/5d52bf9987445b3a6033d66e8644ed77c4d761bd
Stats:     7 lines in 2 files changed: 2 ins; 3 del; 2 mod

8279910: G1: Simplify HeapRegionRemSet::add_reference

Reviewed-by: tschatzl, ayang

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

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



More information about the hotspot-gc-dev mailing list