Richard Henderson <richard.hender...@linaro.org> writes:
> 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.
I fixed this up by using read_vec_element to load the value.
--
Alex Bennée
