RFR: 8299795: Relativize locals in interpreter frames [v2]
Coleen Phillimore
coleenp at openjdk.org
Fri Jan 13 16:51:22 UTC 2023
On Wed, 11 Jan 2023 09:22:03 GMT, Fredrik Bredberg <duke at openjdk.org> wrote:
>> Implementation of relativized locals in interpreter frames for x86. x64, arm, aarch64, ppc64le and riscv.
>> Not relativized locals on zero and s390 but done some changes to cope with the changed generic code.
>> Tested tier1-tier8 on supported platforms. The rest was sanity tested using Qemu, except s390, which was only tested by GitHub Actions.
>
> Fredrik Bredberg has updated the pull request incrementally with one additional commit since the last revision:
>
> Updated some copyright dates.
This looks great. Very nice work!
src/hotspot/cpu/aarch64/continuationFreezeThaw_aarch64.inline.hpp line 306:
> 304: // set relativized locals
> 305: // this line can be changed into an assert when we have fixed the "frame padding problem"
> 306: *f.addr_at(frame::interpreter_frame_locals_offset) = (bottom - 1) - f.fp();
Thank you for this comment. When you file the CR for this problem, can you point to this CR?
src/hotspot/cpu/ppc/continuationFreezeThaw_ppc.inline.hpp line 554:
> 552: inline void ThawBase::set_interpreter_frame_bottom(const frame& f, intptr_t* bottom) {
> 553: // set relativized locals
> 554: // this line can be changed into an assert when we have fixed the "frame padding problem"
Can you file an RFE (or is there already one) to describe this?
-------------
Marked as reviewed by coleenp (Reviewer).
PR: https://git.openjdk.org/jdk/pull/11902
More information about the serviceability-dev
mailing list