> This is the second attempt to fix method `post_meth_exit` to correctly set > state and preserve result. > The related fix here: > https://github.com/openjdk/jdk/commit/b7b64bb6c800b45e32ff37b1b92b5927a3b3fb56 > Hope fix became clarere now. > > There 2 problems in this post_meth_exit: > 1) The result is preserved only if > `state->is_enabled(JVMTI_EVENT_METHOD_EXIT)` however transition in the > JRT_BLOCK_END happens always. So there is a risk of loosing method results in > the interp_only mode. > 2) The method `get_jvmti_thread_state` should be called when thread is in vm > state only. > > The fix adds `post_method_exit_transition` to have single exit point with oop > restoration.
Leonid Mesnik 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 five additional commits since the last revision: - small improvements - Merge branch 'master' of https://github.com/openjdk/jdk into 8365192-2 - comment - comment is fixed. - fix ------------- Changes: - all: https://git.openjdk.org/jdk/pull/27112/files - new: https://git.openjdk.org/jdk/pull/27112/files/77c9c7a3..0bd9c98d Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=27112&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27112&range=01-02 Stats: 997 lines in 48 files changed: 628 ins; 159 del; 210 mod Patch: https://git.openjdk.org/jdk/pull/27112.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/27112/head:pull/27112 PR: https://git.openjdk.org/jdk/pull/27112
