RFR: 8370766: JVM crashes when running compiler/exceptions/TestAccessErrorInCatch.java fails with -XX:+VerifyStack [v2]

[Dean Long]

> The problem is C2 is throwing an exception and then deoptimizing, and the -XX:+VerifyStack logic expects the stack to be empty, match the "before" state if the reexecute flag is set, or match the "after" state.  C2 is using the "before" state, so for correctness it also needs to set the reexecute flag.
> 
> I played around with other approaches, like:
> 1. setting the stack to empty
> 2. adding all the bytecodes that can throw to the list in AbstractInterpreter::bytecode_should_reexecute()
> 3. always setting the reexecute flag in add_safepoint_edges() if must_throw is set
> but in the end I decided to go with the minimal localized low-risk change.

Dean Long has updated the pull request incrementally with one additional commit since the last revision:

  remove extra spaces

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28486/files
  - new: https://git.openjdk.org/jdk/pull/28486/files/a319cc08..8f89b007

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28486&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28486&range=00-01

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

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