Integrated: 8271140: Fix native frame handling in vframeStream::asJavaVFrame()
Andreas Woess
aw at openjdk.java.net
Mon Jul 26 19:50:33 UTC 2021
On Thu, 22 Jul 2021 13:07:46 GMT, Andreas Woess <aw at openjdk.org> wrote:
> Follow-up to https://github.com/openjdk/jdk/pull/4625 ([JDK-8269592](https://bugs.openjdk.java.net/browse/JDK-8269592)) which added support for native frames to `vframeStreamCommon::asJavaVFrame()`. This change was not correct when `asJavaVFrame()` is called for a native frame that is the last frame on the stack (i.e. the first frame in the stream), in which case there's no `_prev_frame` yet. We don't actually need the extended frame information for native frames, so the fix is to just use the vframeStream's `_frame` and `_reg_map` for native frames.
> Extended the test from JDK-8269592 to cover this case.
This pull request has now been integrated.
Changeset: 3aadae20
Author: Andreas Woess <aw at openjdk.org>
Committer: Tom Rodriguez <never at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/3aadae2077e9bf0a5900af79929b679bc6ec62b2
Stats: 78 lines in 2 files changed: 41 ins; 19 del; 18 mod
8271140: Fix native frame handling in vframeStream::asJavaVFrame()
Reviewed-by: dnsimon, kvn, never
-------------
PR: https://git.openjdk.java.net/jdk/pull/4872
More information about the hotspot-compiler-dev
mailing list