On Tue, 23 Aug 2022 11:29:45 GMT, Sacha Coppey <d...@openjdk.org> 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

Reply via email to