On Mon, 20 Feb 2023 09:08:59 GMT, Richard Reingruber <rr...@openjdk.org> wrote:

>> ... and no regression in the other serviceability tests.
>
> 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.

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

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

Reply via email to