On Thu, 23 Jun 2022 15:34:11 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits since the last revision: >> >> - resolved 8286580 review comments; problem-listed test GetSetLocalTest >> under 8286836 >> - Merge branch 'master' into br2 >> - remove _depth == 0 clause from sanity check assert >> - 8286580: serviceability/jvmti/vthread/GetSetLocalTest failed with assert: >> Not supported for heap frames > > src/hotspot/share/prims/jvmtiImpl.cpp line 654: > >> 652: // frame at this point, because we aren't truly in the callee yet. >> 653: // fr.is_heap_frame() is impossible if a continuation is at a >> single step or breakpoint. >> 654: // In such cases the top frames can't be frozen because of an >> agent callback frame. > > "frames" should be "frame". I don't understand the agent callback frame > reference. Removed this sentence as unneeded. ------------- PR: https://git.openjdk.org/jdk19/pull/42
