Skip to content

Commit

Permalink
8308814: extend SetLocalXXX minimal support for virtual threads
Browse files Browse the repository at this point in the history
Reviewed-by: cjplummer
  • Loading branch information
Serguei Spitsyn committed May 27, 2023
1 parent a923634 commit bd113ee
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions src/hotspot/share/prims/jvmti.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5826,9 +5826,8 @@ class C2 extends C1 implements I2 {
The <code>GetLocalXXX</code> functions may be used to retrieve the value of
a local variable contained in the frame of a virtual thread.
The <code>SetLocalXXX</code> functions may be used to set the value of a
local variable in the topmost frame of a virtual thread suspended at a
breakpoint or single step event. An implementation may support setting locals
in other cases.
local variable in the topmost frame of a virtual thread suspended at an event.
An implementation may support setting locals in other cases.
</intro>

<function id="GetLocalObject" num="21">
Expand Down Expand Up @@ -6205,7 +6204,7 @@ class C2 extends C1 implements I2 {
Not a visible frame
</error>
<error id="JVMTI_ERROR_OPAQUE_FRAME">
The thread is a virtual thread and the implementation does not support
The thread is a suspended virtual thread and the implementation does not support
setting the value of locals in the frame of the given depth.
See <internallink id="local">Local Variables</internallink>.
</error>
Expand Down Expand Up @@ -6269,7 +6268,7 @@ class C2 extends C1 implements I2 {
Not a visible frame
</error>
<error id="JVMTI_ERROR_OPAQUE_FRAME">
The thread is a virtual thread and the implementation does not support
The thread is a suspended virtual thread and the implementation does not support
setting the value of locals in the frame of the given depth.
See <internallink id="local">Local Variables</internallink>.
</error>
Expand Down Expand Up @@ -6328,7 +6327,7 @@ class C2 extends C1 implements I2 {
Not a visible frame
</error>
<error id="JVMTI_ERROR_OPAQUE_FRAME">
The thread is a virtual thread and the implementation does not support
The thread is a suspended virtual thread and the implementation does not support
setting the value of locals in the frame of the given depth.
See <internallink id="local">Local Variables</internallink>.
</error>
Expand Down Expand Up @@ -6387,7 +6386,7 @@ class C2 extends C1 implements I2 {
Not a visible frame
</error>
<error id="JVMTI_ERROR_OPAQUE_FRAME">
The thread is a virtual thread and the implementation does not support
The thread is a suspended virtual thread and the implementation does not support
setting the value of locals in the frame of the given depth.
See <internallink id="local">Local Variables</internallink>.
</error>
Expand Down Expand Up @@ -6446,7 +6445,7 @@ class C2 extends C1 implements I2 {
Not a visible frame
</error>
<error id="JVMTI_ERROR_OPAQUE_FRAME">
The thread is a virtual thread and the implementation does not support
The thread is a suspended virtual thread and the implementation does not support
setting the value of locals in the frame of the given depth.
See <internallink id="local">Local Variables</internallink>.
</error>
Expand Down

1 comment on commit bd113ee

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.