[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v5 1/9] softfloat: Implement run-time-configurable
From: |
Leon Alrae |
Subject: |
Re: [Qemu-ppc] [PATCH v5 1/9] softfloat: Implement run-time-configurable meaning of signaling NaN bit |
Date: |
Fri, 29 Apr 2016 14:49:34 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111124 Thunderbird/8.0 |
On 18/04/16 17:03, Aleksandar Markovic wrote:
> -#if SNAN_BIT_IS_ONE
> - return ((uint32_t)(a << 1) >= 0xff800000);
> -#else
> - return ( ( ( a>>22 ) & 0x1FF ) == 0x1FE ) && ( a & 0x003FFFFF );
> -#endif
> + if (status->snan_bit_is_one) {
> + return ((uint32_t)(a << 1) >= 0xFF800000);
> + } else {
> + return (((a >> 22) & 0x1FF) == 0x1FE) && (a & 0x003FFFFF);
Thanks for fixing the style of lines you modified, ...
> - z.sign = float32_val(a)>>31;
> + z.sign = float32_val(a) >> 31;
> z.low = 0;
> - z.high = ( (uint64_t) float32_val(a) )<<41;
> + z.high = ((uint64_t)float32_val(a)) << 41;
... here however I think we usually don't correct the style if the line
wouldn't be touched otherwise. But obviously this is up to FPU Maintainers.
> @@ -2940,7 +2952,8 @@ void helper_msa_fclass_df(CPUMIPSState *env, uint32_t
> df,
> c = update_msacsr(env, CLEAR_FS_UNDERFLOW, 0); \
> \
> if (get_enabled_exceptions(env, c)) { \
> - DEST = ((FLOAT_SNAN ## BITS >> 6) << 6) | c; \
> + DEST = ((FLOAT_SNAN ## BITS(&env->active_tc.msa_fp_status) \
You can use the existing local pointer "status". Similarly in other MSA
macros.
> + >> 6) << 6) | c; \
> @@ -4670,7 +4670,7 @@ static void disas_sparc_insn(DisasContext * dc,
> unsigned int insn)
> TCGv r_const;
>
> gen_address_mask(dc, cpu_addr);
> - tcg_gen_qemu_ld8u(cpu_val, cpu_addr, dc->mem_idx);
> + tcg_gen_qemu_ld8s(cpu_val, cpu_addr, dc->mem_idx);
This change appeared here by mistake, isn't it?
Thanks,
Leon
- [Qemu-ppc] [PATCH v5 6/9] target-mips: Add abs2008 flavor of <ABS|NEG>.<S|D>, (continued)
- [Qemu-ppc] [PATCH v5 6/9] target-mips: Add abs2008 flavor of <ABS|NEG>.<S|D>, Aleksandar Markovic, 2016/04/18
- [Qemu-ppc] [PATCH v5 8/9] target-mips: Clean up position and order of helpers for CVT.<L|W>.<S|D>, Aleksandar Markovic, 2016/04/18
- [Qemu-ppc] [PATCH v5 7/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Aleksandar Markovic, 2016/04/18
- [Qemu-ppc] [PATCH v5 5/9] target-mips: Activate IEEE 274-2008 signaling NaN bit meaning, Aleksandar Markovic, 2016/04/18
- [Qemu-ppc] [PATCH v5 9/9] target-mips: Clean up position of abs2008/nan2008 cases in genfarith(), Aleksandar Markovic, 2016/04/18
- [Qemu-ppc] [PATCH v5 1/9] softfloat: Implement run-time-configurable meaning of signaling NaN bit, Aleksandar Markovic, 2016/04/18
- Re: [Qemu-ppc] [PATCH v5 1/9] softfloat: Implement run-time-configurable meaning of signaling NaN bit,
Leon Alrae <=