Re: [Qemu-devel] [PATCH RFC v0 4/6] target-ppc: add vslv instruction

[David Gibson]

On Wed, Jul 27, 2016 at 12:56:56AM +0530, Nikunj A Dadhania wrote:
> From: Vivek Andrew Sha <address@hidden>
> 
> vslv: Vector Shift Left Variable
> 
> Signed-off-by: Vivek Andrew Sha <address@hidden>
> Signed-off-by: Nikunj A Dadhania <address@hidden>

Reviewed-by: David Gibson <address@hidden>

> ---
>  target-ppc/helper.h     |  1 +
>  target-ppc/int_helper.c | 14 ++++++++++++++
>  target-ppc/translate.c  |  2 ++
>  3 files changed, 17 insertions(+)
> 
> diff --git a/target-ppc/helper.h b/target-ppc/helper.h
> index e93b84b..9703f85 100644
> --- a/target-ppc/helper.h
> +++ b/target-ppc/helper.h
> @@ -211,6 +211,7 @@ DEF_HELPER_3(vslw, void, avr, avr, avr)
>  DEF_HELPER_3(vsld, void, avr, avr, avr)
>  DEF_HELPER_3(vslo, void, avr, avr, avr)
>  DEF_HELPER_3(vsro, void, avr, avr, avr)
> +DEF_HELPER_3(vslv, void, avr, avr, avr)
>  DEF_HELPER_3(vaddcuw, void, avr, avr, avr)
>  DEF_HELPER_3(vsubcuw, void, avr, avr, avr)
>  DEF_HELPER_2(lvsl, void, avr, tl)
> diff --git a/target-ppc/int_helper.c b/target-ppc/int_helper.c
> index bffe8d6..412398f 100644
> --- a/target-ppc/int_helper.c
> +++ b/target-ppc/int_helper.c
> @@ -1708,6 +1708,20 @@ VSL(w, u32, 0x1F)
>  VSL(d, u64, 0x3F)
>  #undef VSL
>  
> +void helper_vslv(ppc_avr_t *r, ppc_avr_t *a, ppc_avr_t *b)
> +{
> +    int i;
> +    unsigned int shift, bytes, size;
> +
> +    size = ARRAY_SIZE(r->u8);
> +    for (i = 0; i < size; i++) {
> +        shift = b->u8[i] & 0x7;             /* extract shift value */
> +        bytes = (a->u8[i] << 8) +             /* extract adjacent bytes */
> +            (((i + 1) < size) ? a->u8[i + 1] : 0);
> +        r->u8[i] = (bytes << shift) >> 8;   /* shift and store result */
> +    }
> +}
> +
>  void helper_vsldoi(ppc_avr_t *r, ppc_avr_t *a, ppc_avr_t *b, uint32_t shift)
>  {
>      int sh = shift & 0xf;
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index 7cf0c8e..473f21a 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -7457,6 +7457,7 @@ GEN_VXFORM(vsraw, 2, 14);
>  GEN_VXFORM(vsrad, 2, 15);
>  GEN_VXFORM(vslo, 6, 16);
>  GEN_VXFORM(vsro, 6, 17);
> +GEN_VXFORM(vslv, 2, 29);
>  GEN_VXFORM(vaddcuw, 0, 6);
>  GEN_VXFORM(vsubcuw, 0, 22);
>  GEN_VXFORM_ENV(vaddubs, 0, 8);
> @@ -10942,6 +10943,7 @@ GEN_VXFORM(vsraw, 2, 14),
>  GEN_VXFORM_207(vsrad, 2, 15),
>  GEN_VXFORM(vslo, 6, 16),
>  GEN_VXFORM(vsro, 6, 17),
> +GEN_VXFORM(vslv, 2, 29),
>  GEN_VXFORM(vaddcuw, 0, 6),
>  GEN_VXFORM(vsubcuw, 0, 22),
>  GEN_VXFORM(vaddubs, 0, 8),

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson

signature.asc
Description: PGP signature