On Mon, 25 Jul 2022 14:38:26 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. It will be
>> tested soon in Native Image as well.
>
> Sacha Coppey has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Avoid using set_destination when call is not jal
src/jdk.internal.vm.ci/share/classes/jdk.vm.ci.riscv64/src/jdk/vm/ci/riscv64/RISCV64Kind.java
line 111:
> 109:
> 110: public boolean isFP() {
> 111: switch(this) {
let's add space after `switch`
-------------
PR: https://git.openjdk.org/jdk/pull/9587