Integrated: 8332130: RISC-V: remove wrong instructions of Vector Crypto Extension

Hamlin Li mli at openjdk.org
Tue May 14 11:30:09 UTC 2024


On Mon, 13 May 2024 08:14:43 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to reivew this simple patch to remove some wrong instrunctions on riscv?
> These instrunctions are wrong in that e.g. take `vror.vx` as example,
> * by definition of spec, it should be `vror.vx vd, vs2, *rs1*, vm`
> * the implementation here, it is indeed `vror_vx(VectorRegister Vd, VectorRegister Vs2, *VectorRegister* Vs1, VectorMask vm = unmasked)`
> 
> Thanks

This pull request has now been integrated.

Changeset: 7ce4a13c
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7ce4a13c0a891e606480e138f4025ffa328a18b3
Stats:     5 lines in 1 file changed: 0 ins; 5 del; 0 mod

8332130: RISC-V: remove wrong instructions of Vector Crypto Extension

Reviewed-by: luhenry, fyang

-------------

PR: https://git.openjdk.org/jdk/pull/19211


More information about the hotspot-compiler-dev mailing list