Re: [PATCH] target/riscv: Fix zfa fleq.d and fltq.d

2023-08-10 Thread Alistair Francis
On Thu, Jul 27, 2023 at 8:50 PM LIU Zhiwei wrote: > > Commit a47842d ("riscv: Add support for the Zfa extension") implemented the > zfa extension. > However, it has some typos for fleq.d and fltq.d. Both of them misused the > fltq.s > helper function. > > Signed-off-by: LIU Zhiwei Thanks! App

Re: [PATCH] target/riscv: Fix zfa fleq.d and fltq.d

2023-07-28 Thread Daniel Henrique Barboza
On 7/27/23 21:39, LIU Zhiwei wrote: Commit a47842d ("riscv: Add support for the Zfa extension") implemented the zfa extension. However, it has some typos for fleq.d and fltq.d. Both of them misused the fltq.s helper function. Signed-off-by: LIU Zhiwei --- Reviewed-by: Daniel Henrique Bar

Re: [PATCH] target/riscv: Fix zfa fleq.d and fltq.d

2023-07-27 Thread Weiwei Li
On 2023/7/28 08:39, LIU Zhiwei wrote: Commit a47842d ("riscv: Add support for the Zfa extension") implemented the zfa extension. However, it has some typos for fleq.d and fltq.d. Both of them misused the fltq.s helper function. Signed-off-by: LIU Zhiwei --- Reviewed-by: Weiwei Li Weiwei

[PATCH] target/riscv: Fix zfa fleq.d and fltq.d

2023-07-27 Thread LIU Zhiwei
Commit a47842d ("riscv: Add support for the Zfa extension") implemented the zfa extension. However, it has some typos for fleq.d and fltq.d. Both of them misused the fltq.s helper function. Signed-off-by: LIU Zhiwei --- target/riscv/insn_trans/trans_rvzfa.c.inc | 4 ++-- 1 file changed, 2 inse