On Tue, 23 Aug 2022 11:29:45 GMT, Sacha Coppey <[email protected]> wrote:
>> This patch adds a partial JVMCI implementation for RISC-V, to allow using
>> the GraalVM Native Image RISC-V LLVM backend, which does not use JVMCI for
>> code emission.
>> It creates the jdk.vm.ci.riscv64 and jdk.vm.ci.hotspot.riscv64 packages, as
>> well as implements a part of jvmciCodeInstaller_riscv64.cpp. To check for
>> correctness, it enables JVMCI code installation tests on RISC-V. More
>> testing is performed in Native Image.
>
> Sacha Coppey has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Fix error when emitting LUI and removed vector registers
src/hotspot/share/runtime/deoptimization.hpp line 345:
> 343: }
> 344: }
> 345: static int trap_request_index(int trap_request) {
Please add a comment as to why `noinline` is required here.
-------------
PR: https://git.openjdk.org/jdk/pull/9587
