Re: [PATCH v2] target/riscv: add support for zmmul extension v0.1

2022-05-30 Thread Alistair Francis
On Tue, May 24, 2022 at 2:54 PM Weiwei Li wrote: > > - includes all multiplication operations for M extension > > Signed-off-by: Weiwei Li > Signed-off-by: Junqiang Wang Do you mind rebasing this on the latest master branch and sending a v3? Alistair > > v2: > * disable M when both M and Zmm

Re: [PATCH v2] target/riscv: add support for zmmul extension v0.1

2022-05-26 Thread Alistair Francis
On Tue, May 24, 2022 at 2:54 PM Weiwei Li wrote: > > - includes all multiplication operations for M extension > > Signed-off-by: Weiwei Li > Signed-off-by: Junqiang Wang Reviewed-by: Alistair Francis Alistair > > v2: > * disable M when both M and Zmmul are enabled > > --- > target/riscv/cp

Re: [PATCH v2] target/riscv: add support for zmmul extension v0.1

2022-05-24 Thread Víctor Colombo
On 24/05/2022 01:53, Weiwei Li wrote: - includes all multiplication operations for M extension Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang v2: * disable M when both M and Zmmul are enabled I tested locally and everything seems to be working LGTM Reviewed-by: Víctor Colombo

[PATCH v2] target/riscv: add support for zmmul extension v0.1

2022-05-23 Thread Weiwei Li
- includes all multiplication operations for M extension Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang v2: * disable M when both M and Zmmul are enabled --- target/riscv/cpu.c | 7 +++ target/riscv/cpu.h | 1 + target/riscv/insn_trans/