Hamlin Li

> What do you mean by wrong? Happy to remove them but we should give some more context. Thanks, update the pr desc to explain why they're wrong.

> I think you mean the `funct3` (`OPIVV` vs `OPIVX`) encoding is wrong? Yes

Sorry for misleading. Thanks @luhenry @RealFYang for your reviewing. /integrate

And a general question about `nativeInst_riscv.cpp` and `macroAssembler_riscv.cpp`. I saw the functions in these 2 files call each other, that make the code a bit mess to me. It's not...

> > And a general question about `nativeInst_riscv.cpp` and `macroAssembler_riscv.cpp`. I saw the functions in these 2 files call each other, that make the code a bit mess to me....

Yes, both riscv and s390x encountered below issue: ``` /github_workspace/sbt-java-module-info /github_workspace Downloading sbt launcher for 1.9.0: From https://repo1.maven.org/maven2/org/scala-sbt/sbt-launch/1.9.0/sbt-launch-1.9.0.jar To /root/.sbt/launchers/1.9.0/sbt-launch.jar Downloading sbt launcher 1.9.0 md5 hash: From https://repo1.maven.org/maven2/org/scala-sbt/sbt-launch/1.9.0/sbt-launch-1.9.0.jar.md5 To /root/.sbt/launchers/1.9.0/sbt-launch.jar.md5...

Seems we still face the same issue.

Hi @odaira , Sorry for disturbing. When avaiable, can you help to update lz4-java with a riscv64 support? riscv64 is now supported [1](https://github.com/lz4/lz4/pull/1298), [2](https://github.com/lz4/lz4/pull/1299) by lz4 itself. Please kindly let...

/author set @XiaohongGong

/contributor add @Hamlin-Li