RFR: 8330253: Remove verify_consistent_lock_order [v8]

Axel Boldt-Christmas aboldtch at openjdk.org
Mon Apr 29 13:52:23 UTC 2024


> The verification added in [JDK-8329757](https://bugs.openjdk.org/browse/JDK-8329757) will not work when deoptimization occurs on a monitorenter bytecode. The locking may be in a transitional state. Because the correct solution is still not obvious and this test is currently only causing false positives, remove the verification for now.
> 
> Redo this verification in [JDK-8331307](https://bugs.openjdk.org/browse/JDK-8331307).

Axel Boldt-Christmas has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision:

 - Merge tag 'jdk-23+20' into JDK-8330164
   
   Added tag jdk-23+20 for changeset 87e864bf
 - Revert verification
 - Revert "repro for JDK-8330253"
   
   This reverts commit 10d70ea18c5bad0627b9eaae88fa8c96e436cb3b.
 - Fix condition
 - Use raw bytecode read for previous bytecode
 - repro for JDK-8330253
 - Whitespace
 - Spelling and typos
 - Handle previous bc being monitorenter
 - Remove implicit conditions
 - ... and 1 more: https://git.openjdk.org/jdk/compare/d73b29e7...d913ecbe

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/18782/files
  - new: https://git.openjdk.org/jdk/pull/18782/files/be0a6883..d913ecbe

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18782&range=07
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18782&range=06-07

  Stats: 58533 lines in 773 files changed: 29910 ins; 25491 del; 3132 mod
  Patch: https://git.openjdk.org/jdk/pull/18782.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18782/head:pull/18782

PR: https://git.openjdk.org/jdk/pull/18782


More information about the hotspot-dev mailing list