RFR: 8296916: RISC-V: Move some small macro-assembler functions to header file [v2]

Aleksey Shipilev shade at openjdk.org
Wed Nov 16 07:39:59 UTC 2022


On Mon, 14 Nov 2022 02:58:20 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Witnessed that there are some small macro-assembler functions located in file macroAssembler_riscv.cpp.
>> These are small functions which mostly contain only a single line of code. We should move them to the
>> corresponding header file so that they have a chance to be inlined.
>> 
>> Testing: Tier1 on linux-riscv64 HiFive unmatched board.
>
> Fei Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review

I would have expected these to be moved to `macroAssembler_riscv.inline.hpp`, to be honest.

But this is okay as well.

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

PR: https://git.openjdk.org/jdk/pull/11130Marked as reviewed by shade (Reviewer).


More information about the hotspot-dev mailing list