RFR: 8337797: Additional ExternalAddress cleanup [v2]
Vladimir Kozlov
kvn at openjdk.org
Tue Aug 6 17:00:35 UTC 2024
On Tue, 6 Aug 2024 13:48:57 GMT, Andrew Dinn <adinn at openjdk.org> wrote:
>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> riscv update
>
> src/hotspot/cpu/x86/stubGenerator_x86_64_arraycopy.cpp line 218:
>
>> 216: __ lea(end_from, Address(from, count, sf, 0));
>> 217: if (NOLp == nullptr) {
>> 218: RuntimeAddress no_overlap(no_overlap_target);
>
> I think this is actually a jump within the same buffer. Can we use an AddressLiteral here with reloc_none and rely on this being encoded with a PC-relative jump?
`relic_none` is treated as unreachable and will force using lea():
https://github.com/openjdk/jdk/blob/master/src/hotspot/cpu/x86/assembler_x86.cpp#L12915
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20470#discussion_r1705853432
More information about the hotspot-dev
mailing list