On Fri, 29 Aug 2025 19:11:04 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> The void `JvmtiExport::post_method_exit(JavaThread* thread, Method* method, >> frame current_frame) `calculates >> `bool exception_exit = state->is_exception_detected() && >> !state->is_exception_caught();` >> to find if method exit normally or by exception. >> However, JvmtiExport::post_method_exit( method is not called at all in the >> case of exception. See >> `void JvmtiExport::notice_unwind_due_to_exception(JavaThread *thread, >> Method* method, address location, oop exception, bool in_handler_frame)` >> where post_method_exit_inner is called directly. >> >> The `exception_exit` is true when exception is processed and the current >> method is called in the middle of stack unwinding. >> >> >> The fix was a part of >> https://github.com/openjdk/jdk/pull/26713 >> for >> https://bugs.openjdk.org/browse/JDK-8365192 > > Leonid Mesnik has updated the pull request incrementally with two additional > commits since the last revision: > > - Apply suggestions from code review > > Co-authored-by: Patricio Chilano Mateo <patricio.chilano.ma...@oracle.com> > - Update > test/hotspot/jtreg/serviceability/jvmti/events/MethodExit/PendingException/libTestMethodExitWithPendingException.cpp > > Co-authored-by: Patricio Chilano Mateo <patricio.chilano.ma...@oracle.com> Okay with me. Thanks ------------- Marked as reviewed by dholmes (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/26886#pullrequestreview-3183671829