On Fri, 13 Mar 2026 11:57:55 GMT, Axel Boldt-Christmas <[email protected]>
wrote:
>> Ivan Walulya has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> cleanup the test
>
> test/hotspot/jtreg/runtime/valhalla/inlinetypes/UnsafeTest.java line 103:
>
>> 101: Value3 v3 = new Value3(v2, List.of("Value3"));
>> 102: long off_o = U.objectFieldOffset(Value3.class, "o");
>> 103: long off_v = U.objectFieldOffset(Value3.class, "v2");
>
> Maybe change this to `off_v2`.
>
> _It is the same offset as the one added below._
> https://github.com/openjdk/valhalla/pull/2213/changes#diff-4067ce8ee796cd3702d2d8d69e847c6a5f4446613c87350b8e9bb422840d7489R136
Fixed.
Thanks for reviewing!
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2213#discussion_r2930908139
