RFR: 8344356: Aarch64: implement -XX:+VerifyActivationFrameSize
Andrew Haley
aph at openjdk.org
Tue Nov 19 09:23:50 UTC 2024
On Sun, 17 Nov 2024 09:06:37 GMT, SendaoYan <syan at openjdk.org> wrote:
> Hi all,
> Currently on aarch64 platform the debug VM option `-XX:+VerifyActivationFrameSize` is Unimplemented. I want to implement `-XX:+VerifyActivationFrameSize` to make JVM easier to diagnose. Only effect debug build plus with option `-XX:+VerifyActivationFrameSize`, the change has been verified locally, the risk is low.
>
> Additional testing
>
> - [ ] jtreg tests(include tier1/2/3 etc.) on linux-aarch64 release build
> - [ ] jtreg tests(include tier1/2/3 etc.) on linux-aarch64 fastdebug build
src/hotspot/cpu/aarch64/interp_masm_aarch64.cpp line 399:
> 397: sub(rscratch2, rfp, esp);
> 398: int32_t min_frame_size = (frame::link_offset - frame::interpreter_frame_mirror_offset) * wordSize;
> 399: cmpw(rscratch2, min_frame_size);
Here too.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22181#discussion_r1847963527
More information about the hotspot-compiler-dev
mailing list