[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