RFR: 8299795: Relativize locals in interpreter frames [v4]

Fredrik Bredberg duke at openjdk.org
Wed Jan 18 15:49:16 UTC 2023


On Wed, 18 Jan 2023 05:07:38 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> My intention was to show how the contents of R18_locals (the absolute address to locals) can be recreated by using R12_scratch2 as an index from the frame pointer. Before we stored R18_locals in the stackframe, now we store R12_scratch2. Does that make more sense?
>
> The code uses &sp()[R12_scratch2]. I think your comment is confusing. Please remove it or write something comprehensive. E.g. save R18_locals in compressed format relative to the new top frame's FP (= current SP).

See what you mean, the R1_SP being the frame pointer is confusing. So how about changing the comment into:
"// Store relativized R18_locals, see frame::interpreter_frame_locals()."
Would that work for you? If not I'll just remove it.

-------------

PR: https://git.openjdk.org/jdk/pull/11902


More information about the serviceability-dev mailing list