Re: [Qemu-devel] [PATCH 12/14] Add vct{u,s}xs instructions

[Aurelien Jarno]

On Sun, Feb 08, 2009 at 02:40:33PM -0800, Nathan Froyd wrote:
> Use round-to-zero mode for conversion, convert to int64_t in all cases,
> and properly set VSCR_SAT for negative values in vctuxs.
> 
> Signed-off-by: Nathan Froyd <address@hidden>
> ---
>  target-ppc/helper.h    |    2 ++
>  target-ppc/op_helper.c |   27 +++++++++++++++++++++++++++
>  target-ppc/translate.c |    2 ++
>  3 files changed, 31 insertions(+), 0 deletions(-)

Thanks, applied.

> diff --git a/target-ppc/helper.h b/target-ppc/helper.h
> index 089955b..3ee04b2 100644
> --- a/target-ppc/helper.h
> +++ b/target-ppc/helper.h
> @@ -253,6 +253,8 @@ DEF_HELPER_2(vrfip, void, avr, avr)
>  DEF_HELPER_2(vrfiz, void, avr, avr)
>  DEF_HELPER_3(vcfux, void, avr, avr, i32)
>  DEF_HELPER_3(vcfsx, void, avr, avr, i32)
> +DEF_HELPER_3(vctuxs, void, avr, avr, i32)
> +DEF_HELPER_3(vctsxs, void, avr, avr, i32)
>  
>  DEF_HELPER_1(efscfsi, i32, i32)
>  DEF_HELPER_1(efscfui, i32, i32)
> diff --git a/target-ppc/op_helper.c b/target-ppc/op_helper.c
> index d40caf0..e4c446d 100644
> --- a/target-ppc/op_helper.c
> +++ b/target-ppc/op_helper.c
> @@ -2288,6 +2288,33 @@ void helper_vcmpbfp_dot (ppc_avr_t *r, ppc_avr_t *a, 
> ppc_avr_t *b)
>      vcmpbfp_internal(r, a, b, 1);
>  }
>  
> +#define VCT(suffix, satcvt, element)                                    \
> +    void helper_vct##suffix (ppc_avr_t *r, ppc_avr_t *b, uint32_t uim)  \
> +    {                                                                   \
> +        int i;                                                          \
> +        int sat = 0;                                                    \
> +        float_status s = env->vec_status;                               \
> +        set_float_rounding_mode(float_round_to_zero, &s);               \
> +        for (i = 0; i < ARRAY_SIZE(r->f); i++) {                        \
> +            if (float32_is_nan(b->f[i]) ||                              \
> +                float32_is_signaling_nan(b->f[i])) {                    \
> +                r->element[i] = 0;                                      \
> +            } else {                                                    \
> +                float64 t = float32_to_float64(b->f[i], &s);            \
> +                int64_t j;                                              \
> +                t = float64_scalbn(t, uim, &s);                         \
> +                j = float64_to_int64(t, &s);                            \
> +                r->element[i] = satcvt(j, &sat);                        \
> +            }                                                           \
> +        }                                                               \
> +        if (sat) {                                                      \
> +            env->vscr |= (1 << VSCR_SAT);                               \
> +        }                                                               \
> +    }
> +VCT(uxs, cvtsduw, u32)
> +VCT(sxs, cvtsdsw, s32)
> +#undef VCT
> +
>  void helper_vmaddfp (ppc_avr_t *r, ppc_avr_t *a, ppc_avr_t *b, ppc_avr_t *c)
>  {
>      int i;
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index a722f65..4d5b45a 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -6521,6 +6521,8 @@ GEN_VXFORM_UIMM(vsplth, 6, 9);
>  GEN_VXFORM_UIMM(vspltw, 6, 10);
>  GEN_VXFORM_UIMM(vcfux, 5, 12);
>  GEN_VXFORM_UIMM(vcfsx, 5, 13);
> +GEN_VXFORM_UIMM(vctuxs, 5, 14);
> +GEN_VXFORM_UIMM(vctsxs, 5, 15);
>  
>  GEN_HANDLER(vsldoi, 0x04, 0x16, 0xFF, 0x00000400, PPC_ALTIVEC)
>  {
> -- 
> 1.6.0.5
> 
> 
> 
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
address@hidden                 http://www.aurel32.net