Fei Yang

> > > > Here is the basic support for RISC-V: https://cr.openjdk.java.net/~shade/8291555/riscv-patch-1.patch > > > > -- I adapted this from AArch64 changes, and tested it very lightly. @RealFYang, can...

> > > > > > Here is the basic support for RISC-V: https://cr.openjdk.java.net/~shade/8291555/riscv-patch-1.patch > > > > > > -- I adapted this from AArch64 changes, and tested it...

@rkennke : Could you please add this follow-up fix for RISC-V? I can build fastdebug on HiFive Unmatched board with this fix now and run non-trivial benchmark workloads. I will...

> > I think you mean the `funct3` (`OPIVV` vs `OPIVX`) encoding is wrong? > > Yes From the RVV spec [1], the `funct3` encoding for `OPIVX` is 0b100, which...

> @RealFYang the `.vx` variant expect a **scalar** register while our `vandn_vx` takes a **vector** register. If we had a use for `vandn_vx` (or any of the other removed instructions),...

> Yes, but it's a long term job, as you need to free a register in many cases. (in non-call sites places) All callsites should be easy to change as...

> Hi @RealFYang @luhenry, do you think it is necessary to back-port this commit to jdk19u branch? Thanks! @XieJiSS : Are you using jdk19u and affected by this issue? AFAIK,...

@Hamlin-Li: Thanks for the quick update. Considering that saving/restoring for FRM could be expensive, I do wonder if we could gather some performance numbers before we go. I see people...

So now `MacroAssembler::rt_call` is effectively this after this PR change: ``` void MacroAssembler::rt_call(address dest, Register tmp) { RuntimeAddress target(dest); relocate(target.rspec(), [&] { int32_t offset; la(tmp, target.target(), offset); jalr(tmp, offset); });...

> RuntimeAddress target(dest); Yeah, I know what you mean. As you see, the `address` passed to `rt_call` is used to construct a `RuntimeAddress` in this function which is also an...