On Sun, 21 Jun 2026 05:14:47 GMT, SendaoYan <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review: moved to the right place the line which sets the objectCountMax
>
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/IterateOverReachableObjects/iterreachobj002/iterreachobj002.cpp
> line 354:
>
>> 352: /* This fragment is needed to stress test execution with
>> extra GC's. */
>> 353: for (int gcCount = 0; gcCount < 5; gcCount++) {
>> 354: NSK_DISPLAY1("Calling ForceGarbageCollection #%d before
>> saving objectCountMax\n", gcCount + 1);
>
> The message says the GC runs before saving objectCountMax, but the code does
> the opposite: it saves objectCountMax first, then forces GC.
Good catch, thanks! Fixed now.
It was an incorrect last minute update to make diffs simpler.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31585#discussion_r3456567649
