[lworld] RFR: 8361496: [lworld] Treating interfaces as isValue() leads to failed assert
Chen Liang
liach at openjdk.org
Mon Jul 7 22:49:10 UTC 2025
Reviewed a few sites of isValue() usages that cause immediate trouble with bootstrap and basic tests. Factor out `ValueClass::isValueObjectCompatible` and `isValueObjectInstance` to represent the two states of value classes: field types that can hold value objects, and field types that definitely hold value objects.
I think we still need a more throughout review for all `isValue` usages, such as those in MethodHandle or VarHandle. Preliminary testing hasn't revealed anything yet.
Testing: tier 1-3, seems to have no new failures
-------------
Commit messages:
- More fixes
- 8361496: [lworld] Treating interfaces as isValue() leads to failed assert
Changes: https://git.openjdk.org/valhalla/pull/1504/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1504&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8361496
Stats: 130 lines in 7 files changed: 101 ins; 11 del; 18 mod
Patch: https://git.openjdk.org/valhalla/pull/1504.diff
Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1504/head:pull/1504
PR: https://git.openjdk.org/valhalla/pull/1504
More information about the valhalla-dev
mailing list