RFR: 8257143: Enable JVMCI code installation tests on AArch64
Nick Gasson
ngasson at openjdk.java.net
Tue Dec 1 12:14:55 UTC 2020
On Mon, 30 Nov 2020 19:45:10 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> This set of jtreg tests test JVMCI code installation independently of
>> Graal. Currently they only run on x86 as the minimal assembler required
>> is only implemented for that platform. This patch implements the
>> TestAssembler for AArch64 to ensure JVMCI test coverage even if the
>> Graal embedded in OpenJDK is disabled/removed.
>
> src/hotspot/cpu/aarch64/relocInfo_aarch64.hpp line 35:
>
>> 33: offset_unit = 1,
>> 34: // Must be at least 1 for RelocInfo::narrow_oop_in_const.
>> 35: format_width = 1
>
> Did you hit an issue with = 0? Yes, it should be 1 for 64-bit. But I surprise nobody noticed this until now. How we handled embedded narrow_oop_in_const before?
The assertion `assert((format & format_mask) == format, "wrong format");` fails in `relocInfo::check_format_and_offset()`. That comes from the call in `jvmciCodeInstaller.cpp` line 866. I don't know why we don't see this with regular Graal usage. Maybe it doesn't use that relocation type on AArch64?
-------------
PR: https://git.openjdk.java.net/jdk/pull/1475
More information about the hotspot-compiler-dev
mailing list