Integrated: 8281637: Remove unused VerifyOption_G1UseNextMarking

Stefan Johansson sjohanss at openjdk.java.net
Mon Feb 14 09:07:11 UTC 2022


On Fri, 11 Feb 2022 11:25:56 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

> Please review this change to remove an unused verification option.
> 
> **Summary**
> In G1 there have been three different `VerifyOption` that can be passed to verification to signal how to determine liveness. One of these options, `VerifyOption_G1UseNextMarking`,  has been unused for quite some time and this change removes it. When removing it the `G1CollectedHeap::is_obj_ill(...)` functions are more or less unused, so I removed those as well and inlined the checks in the only place where it is really used `G1CMIsAliveClosure::do_object_b(...)`
> 
> **Testing**
> 
> - [x] Mach5
> - [x] Local verification testing

This pull request has now been integrated.

Changeset: 2632d40d
Author:    Stefan Johansson <sjohanss at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2632d40dfc9f681e53fe04d32b6380ffb4eeb88c
Stats:     66 lines in 9 files changed: 13 ins; 48 del; 5 mod

8281637: Remove unused VerifyOption_G1UseNextMarking

Reviewed-by: tschatzl, iwalulya

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

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



More information about the hotspot-gc-dev mailing list