On Fri, 13 Feb 2026 14:25:54 GMT, Paul Hübner <[email protected]> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fixed gp name
>
> test/hotspot/jtreg/vmTestbase/gc/ArrayJuggle/Juggle3.java line 27:
> 
>> 25:  * VM Testbase keywords: [gc, stress, stressopt, nonconcurrent]
>> 26:  */
>> 27: /* @test id=BooleanObjArr_medium   @key stress randomness @library 
>> /vmTestbase /test/lib @run main/othervm -XX:+HeapDumpOnOutOfMemoryError 
>> -Xlog:gc=debug:gc.log gc.ArrayJuggle.Juggle3 -gp BooleanObjArr        -ms 
>> medium */
> 
> Do we have the resources to run these boxed arrays when not in preview mode? 
> If not, we might want to `@require java.enablePreview` and `@require 
> !java.enablePreview`.

Let run them for now for simplicity.
The '@enablePreview' now might cause issues with compilation, so I don't want 
to use it.

> test/hotspot/jtreg/vmTestbase/nsk/share/gc/gp/GarbageUtils.java line 402:
> 
>> 400:                     return new BooleanObjArrayProducer();
>> 401:                 else if (id.equals("IntegerObjArr"))
>> 402:                     return new IntegerObjArrayProducer();
> 
> Sanity question: do we also need to consider `h_IntegerObjArr`?

No, the 'h_IntegerObjArr' name of testcase while gp `option`  is 
"hashed(IntegerObjArr)". It is parsed correctly.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2099#discussion_r2804839719
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2099#discussion_r2804844737

Reply via email to