[lworld] RFR: 8370509: [lworld] EnableValhalla is not needed for sync
Coleen Phillimore
coleenp at openjdk.org
Thu Oct 23 20:47:56 UTC 2025
I took out EnableValhalla as a conditional test in the ObjectSynchronizer code. If the inline type bit is set in the markWord or the klass is InlineKlass, we can't synchronize on the object.
I added a test but it was sort of a duplicate of all runtime/valhalla/inlinetypes/MonitorEnterTest.java and duplicates all tests that do locking so I didn't add a test.
Testing with tier1-4, still in progress.
-------------
Commit messages:
- 8370509: [lworld] EnableValhalla is not needed for sync
Changes: https://git.openjdk.org/valhalla/pull/1699/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1699&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8370509
Stats: 18 lines in 3 files changed: 0 ins; 3 del; 15 mod
Patch: https://git.openjdk.org/valhalla/pull/1699.diff
Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1699/head:pull/1699
PR: https://git.openjdk.org/valhalla/pull/1699
More information about the valhalla-dev
mailing list