On Thu, 3 Jul 2025 18:41:21 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: (1) remove vthread specific clarifications; unify GetLocal* and >> SetLocal* with other functions > > src/hotspot/share/prims/jvmti.xml line 2991: > >> 2989: <error id="JVMTI_ERROR_OPAQUE_FRAME"> >> 2990: The implementation is unable to pop this frame >> 2991: (e.g. called or calling method is a native method). > > Directly below we have special OPAQUE_FRAME language for virtual threads. Do > we want to get rid of it since it is now covered by the above language? Note > there is also language directly above that discusses suspended virtual > threads. Thank you for the suggestion. Fixed now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26111#discussion_r2193968014
