[lworld] RFR: 8367263: [lworld] fix compiler/valhalla/inlinetypes/TestIntrinsics.java after current merges of JDK-8352737 and JDK-8366705

Marc Chevalier mchevalier at openjdk.org
Wed Sep 10 06:55:31 UTC 2025


TEST64_LAYOUT = U.arrayLayout(TEST64_ARRAY.getClass());

is not correct anymore since [JDK-8366705](https://bugs.openjdk.org/browse/JDK-8366705) as the layout is no longer a class property, but an instance property (if I understood right)

Weirdly enough, code like
 ```
U.arrayBaseOffset(TEST64_ARRAY.getClass());
U.arrayIndexScale(TEST64_ARRAY.getClass());

got also their `.getClass()` removed in [JDK-8366705](https://bugs.openjdk.org/browse/JDK-8366705):
https://github.com/openjdk/valhalla/blob/0817750d434463a606523ac03b5461802f0328e1/test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestIntrinsics.java#L650-L654
but there the compiler wasn't yelling at me. Is it overloaded and it works with instance or class, and the instance overload will just query the class. Not sure... I'm still removing it!

Thanks,
Marc

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

Commit messages:
 - Adapt assert for -XX:-PreloadClasses
 - Fix after JDK-8366705

Changes: https://git.openjdk.org/valhalla/pull/1559/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1559&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8367263
  Stats: 16 lines in 1 file changed: 1 ins; 0 del; 15 mod
  Patch: https://git.openjdk.org/valhalla/pull/1559.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1559/head:pull/1559

PR: https://git.openjdk.org/valhalla/pull/1559


More information about the valhalla-dev mailing list