On Wed, 7 Jan 2026 18:15:09 GMT, Coleen Phillimore <[email protected]> wrote:
>> Frederic Parain has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Fix test configurations names and options
>
> test/hotspot/jtreg/runtime/valhalla/inlinetypes/field_layout/NullMarkersTest.java
> line 368:
>
>> 366: break;
>> 367: case "64_COOP_CCP_COH":
>> 368: compressedOopsArg = "-XX:-UseCompressedOops";
>
> This should be +UseCompressedOops and the one below should be
> -UseCompressedOops.
Thank you for catching that.
Fixed
> test/hotspot/jtreg/runtime/valhalla/inlinetypes/field_layout/ValueFieldInheritanceTest.java
> line 226:
>
>> 224:
>> 225: switch(args[0]) {
>> 226: case "0":
>
> Should be "32".
Fixed
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1862#discussion_r2669616210
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1862#discussion_r2669616517
