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

[Alistair Francis]

On Thu, Jul 27, 2023 at 8:50 PM LIU Zhiwei <zhiwei_...@linux.alibaba.com> 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 <zhiwei_...@linux.alibaba.com>

Thanks!

Applied to riscv-to-apply.next after adding a `Fixes` tag

Alistair

> ---
>  target/riscv/insn_trans/trans_rvzfa.c.inc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target/riscv/insn_trans/trans_rvzfa.c.inc 
> b/target/riscv/insn_trans/trans_rvzfa.c.inc
> index 2c715af3e5..0fdd2698f6 100644
> --- a/target/riscv/insn_trans/trans_rvzfa.c.inc
> +++ b/target/riscv/insn_trans/trans_rvzfa.c.inc
> @@ -470,7 +470,7 @@ bool trans_fleq_d(DisasContext *ctx, arg_fleq_d *a)
>      TCGv_i64 src1 = get_fpr_hs(ctx, a->rs1);
>      TCGv_i64 src2 = get_fpr_hs(ctx, a->rs2);
>
> -    gen_helper_fltq_s(dest, cpu_env, src1, src2);
> +    gen_helper_fleq_d(dest, cpu_env, src1, src2);
>      gen_set_gpr(ctx, a->rd, dest);
>      return true;
>  }
> @@ -485,7 +485,7 @@ bool trans_fltq_d(DisasContext *ctx, arg_fltq_d *a)
>      TCGv_i64 src1 = get_fpr_hs(ctx, a->rs1);
>      TCGv_i64 src2 = get_fpr_hs(ctx, a->rs2);
>
> -    gen_helper_fltq_s(dest, cpu_env, src1, src2);
> +    gen_helper_fltq_d(dest, cpu_env, src1, src2);
>      gen_set_gpr(ctx, a->rd, dest);
>      return true;
>  }
> --
> 2.17.1
>
>