RFR: 8281637: Remove unused VerifyOption_G1UseNextMarking [v2]
Stefan Johansson
sjohanss at openjdk.java.net
Mon Feb 14 09:00:48 UTC 2022
> 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
Stefan Johansson has updated the pull request incrementally with two additional commits since the last revision:
- Update src/hotspot/share/gc/g1/g1ConcurrentMark.inline.hpp
Co-authored-by: Thomas Schatzl <59967451+tschatzl at users.noreply.github.com>
- Update src/hotspot/share/gc/g1/g1ConcurrentMark.inline.hpp
Co-authored-by: Thomas Schatzl <59967451+tschatzl at users.noreply.github.com>
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7440/files
- new: https://git.openjdk.java.net/jdk/pull/7440/files/11e02017..d1439e42
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7440&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7440&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.java.net/jdk/pull/7440.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7440/head:pull/7440
PR: https://git.openjdk.java.net/jdk/pull/7440
More information about the hotspot-gc-dev
mailing list