RFR: 8346143: add ClearAllFramePops function to speedup debugger single stepping in some cases [v2]

[Serguei Spitsyn]

On Tue, 17 Dec 2024 03:20:20 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   added extra check to post_method_exit_inner before clear_frame_pop to avoid assert
>
> src/hotspot/share/interpreter/interpreterRuntime.cpp line 580:
> 
>> 578:   // notify debugger of an exception catch
>> 579:   // (this is good for exceptions caught in native methods as well)
>> 580:   if (JvmtiExport::can_post_on_exceptions() || JvmtiExport::can_post_frame_pop()) {
> 
> This doesn't seem like it is related to ClearAllFramePops.

Yes, it does not relate to the `ClearAllFramePops` directly. But it is needed for correctness of the `NotifyFramePop`. I do not feel it is worth to separate this change.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22744#discussion_r1888946592