[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