On Tue, 17 Nov 2020 14:51:23 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
>> Thomas Schatzl has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - sjohanss review >> - Remove code that "activates" dormant objects as now all active objects >> are reachable via the root object array > > src/hotspot/share/memory/heapShared.cpp line 662: > >> 660: if (!FLAG_IS_DEFAULT(VerifyArchivedFields)) { >> 661: // If this -XX:+VerifyArchivedFields is specified on the >> command-line, do extra >> 662: // checks. > > s/this// I think we can keep the code and change the comments to the following: // If VerifyArchivedFields has a non-default value (e.g., specified on the command-line), do // more expensive checks. ------------- PR: https://git.openjdk.java.net/jdk/pull/1163