On Fri, 17 Feb 2023 14:47:56 GMT, Johannes Bechberger <d...@openjdk.org> wrote:

>> Extends the existing AsyncGetCallTrace test case and fixes the issue.
>
> Johannes Bechberger has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Revert previous change

src/hotspot/cpu/x86/frame_x86.cpp line 75:

> 73:   // interpreted -> interpreted calls that go through a method handle 
> linker,
> 74:   // since those pop the last argument (the appendix) from the stack.
> 75:   if (!thread->is_in_full_stack_checked(unextended_sp)) {

The condition can remain stricter. The following should work too. Could you 
please check?
Suggestion:

  if (!thread->is_in_stack_range_incl(unextended_sp + 
Interpreter::stackElementSize, sp)) {

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

PR: https://git.openjdk.org/jdk/pull/12535

Reply via email to