On Mon, 20 Feb 2023 09:00:28 GMT, Johannes Bechberger <d...@openjdk.org> wrote:
>> 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)) {
>
> The sanity test worked :)
... and no regression in the other serviceability tests.
-------------
PR: https://git.openjdk.org/jdk/pull/12535
