Hi Severin,
On 2/11/20 10:04 AM, Severin Gehwolf wrote:
Updated webrev:
Full: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-823/10/webrev/
incremental:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-823/10/incremental/webrev/
Thanks for updating this. This patch looks okay in ge
Hi Patricio,
> > I'm really glad you noticed the problematic nesting. This seems to be a
general issue: currently a
> > handshake cannot be nested in a vm operation. Maybe it should be asserted
in the
> > Handshake::execute() methods that they are not called by the vm thread
evaluating a
Hi Richard,
On 2/14/20 9:58 AM, Reingruber, Richard wrote:
Hi Patricio,
thanks for having a look.
> I’m only commenting on the handshake changes.
> I see that operation VM_EnterInterpOnlyMode can be called inside
> operation VM_SetFramePop which also allows nested operations. Here is
Hi Patricio,
thanks for having a look.
> I’m only commenting on the handshake changes.
> I see that operation VM_EnterInterpOnlyMode can be called inside
> operation VM_SetFramePop which also allows nested operations. Here is a
> comment in VM_SetFramePop definition:
>
> // Nested