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

[Serguei Spitsyn]

On Tue, 17 Dec 2024 21:38:28 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> 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.
>
> So this is a long standing bug hidden by the fact that if `can_post_frame_pop()` is true, then so is `can_post_on_exceptions()`:
> 
> 
>   JvmtiExport::set_can_post_on_exceptions(
>     avail.can_generate_exception_events ||
>     avail.can_generate_frame_pop_events ||
>     avail.can_generate_method_exit_events);
> 
>   JvmtiExport::set_can_post_frame_pop(avail.can_generate_frame_pop_events);
> 
> 
> Or you could argue that the existing code is correct because it already captures `can_post_frame_pop()`.

Good check, thanks! Rolled back this change. I remember that I've added this because some of the tests was failing. Most likely, I had made a wrong conclusion.

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

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