On Mon, 25 Jul 2022 14:38:26 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. 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

Reply via email to