On Mon, 20 Feb 2023 09:11:38 GMT, Johannes Bechberger <d...@openjdk.org> wrote:

>> Probably it's better to adapt the limit though (sorry).
>> Problem with this version is that it allows `unextended_sp` outside the 
>> address range reserved for the stack. Let's see what other reviewers think.
>> Suggestion:
>> 
>>   if (!thread->is_in_stack_range_incl(unextended_sp, sp - 
>> Interpreter::stackElementSize)) {
>
> Good point, you're right. I could have recognized this myself.

Tests work as expected.

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

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

Reply via email to