RFR: JDK-8283140: Remove unused encoding classes/operands from x86_64.ad [v3]

Dean Long dlong at openjdk.org
Wed Nov 8 07:16:59 UTC 2023


On Fri, 27 Oct 2023 21:32:46 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

>> These encoding classes became dead code when the instructions' definitions were patched to use the MacroAssembler.
>> 
>> Tested with tier 1-3 on Linux, Windows & MacOS.
>
> Cesar Soares Lucas has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix typo

src/hotspot/cpu/x86/x86_64.ad line 1839:

> 1837:                                                   : static_call_Relocation::spec(method_index);
> 1838:       cbuf.relocate(cbuf.insts_mark(), rspec, RELOC_DISP32);
> 1839:       cbuf.insts()->emit_int32((int) ($meth$$method - ((intptr_t) cbuf.insts_end()) - 4));

Suggestion:

      __ call(AddressLiteral(CAST_FROM_FN_PTR(address, $meth$$method), rtype), noreg);

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16319#discussion_r1386108123


More information about the hotspot-compiler-dev mailing list