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

Reply via email to