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

Reply via email to