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