On 02/08/2018 09:31 AM, Alex Bennée wrote:
> This includes FMOV, FABS, FNEG, FSQRT and FRINT[NPMZAXI]. We re-use
> existing helpers to achieve this.
>
> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
> ---
> target/arm/translate-a64.c | 72
> ++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 72 insertions(+)
>
> diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
> index 92adf43a89..265bfb14d0 100644
> --- a/target/arm/translate-a64.c
> +++ b/target/arm/translate-a64.c
> @@ -4508,6 +4508,66 @@ static void disas_fp_csel(DisasContext *s, uint32_t
> insn)
> tcg_temp_free_i64(t_true);
> }
>
> +/* Floating-point data-processing (1 source) - half precision */
> +static void handle_fp_1src_half(DisasContext *s, int opcode, int rd, int rn)
> +{
> + TCGv_ptr fpst = NULL;
> + TCGv_i32 tcg_op;
> + TCGv_i32 tcg_res;
> +
> + tcg_op = read_fp_sreg(s, rn);
> + tcg_res = tcg_temp_new_i32();
> +
> + switch (opcode) {
> + case 0x0: /* FMOV */
> + tcg_gen_mov_i32(tcg_res, tcg_op);
> + break;
> + case 0x1: /* FABS */
> + gen_helper_advsimd_absh(tcg_res, tcg_op);
> + break;
> + case 0x2: /* FNEG */
> + tcg_gen_xori_i32(tcg_res, tcg_op, 0x8000);
> + break;
> + case 0x3: /* FSQRT */
> + gen_helper_sqrt_f16(tcg_res, tcg_op, cpu_env);
> + break;
> + case 0x8: /* FRINTN */
> + case 0x9: /* FRINTP */
> + case 0xa: /* FRINTM */
> + case 0xb: /* FRINTZ */
> + case 0xc: /* FRINTA */
> + {
> + TCGv_i32 tcg_rmode = tcg_const_i32(arm_rmode_to_sf(opcode & 7));
> + fpst = get_fpstatus_ptr(true);
> +
> + gen_helper_set_rmode(tcg_rmode, tcg_rmode, fpst);
> + gen_helper_advsimd_rinth(tcg_res, tcg_op, fpst);
> +
> + gen_helper_set_rmode(tcg_rmode, tcg_rmode, fpst);
> + tcg_temp_free_i32(tcg_rmode);
> + break;
> + }
> + case 0xe: /* FRINTX */
> + fpst = get_fpstatus_ptr(true);
> + gen_helper_advsimd_rinth_exact(tcg_res, tcg_op, fpst);
> + break;
> + case 0xf: /* FRINTI */
> + fpst = get_fpstatus_ptr(true);
> + gen_helper_advsimd_rinth(tcg_res, tcg_op, fpst);
> + break;
> + default:
> + abort();
> + }
> +
> + write_fp_sreg(s, rd, tcg_res);
Some of these helpers will zero-extend from 16 bits, but at least a few won't
-- notably fmov and fneg. I wonder if it wouldn't be best to have a
write_fp_hreg.
r~
