RFR: 8278874: tighten VerifyStack constraints [v5]

Dean Long dlong at openjdk.org
Thu Jul 24 19:34:15 UTC 2025


> The VerifyStack logic in Deoptimization::unpack_frames() attempts to check the expression stack size of the interpreter frame against what GenerateOopMap computes.  To do this, it needs to know if the state at the current bci represents the "before" state, meaning the bytecode will be reexecuted, or the "after" state, meaning we will advance to the next bytecode.  The old code didn't know how to determine exactly what state we were in, so it checked both.  This PR cleans that up, so we only have to compute the oopmap once.  It also removes old SPARC support.

Dean Long has updated the pull request incrementally with two additional commits since the last revision:

 - Update src/hotspot/share/runtime/vframeArray.cpp
   
   Co-authored-by: Manuel Hässig <manuel at haessig.org>
 - Update src/hotspot/share/runtime/vframeArray.cpp
   
   Co-authored-by: Manuel Hässig <manuel at haessig.org>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/26121/files
  - new: https://git.openjdk.org/jdk/pull/26121/files/c93ca9e1..54919be0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=26121&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26121&range=03-04

  Stats: 3 lines in 1 file changed: 2 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/26121.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26121/head:pull/26121

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


More information about the hotspot-compiler-dev mailing list