[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v2 33/67] target/arm: Implement SVE reverse within
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH v2 33/67] target/arm: Implement SVE reverse within elements |
Date: |
Fri, 23 Feb 2018 15:50:11 +0000 |
On 17 February 2018 at 18:22, Richard Henderson
<address@hidden> wrote:
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target/arm/helper-sve.h | 14 ++++++++++++++
> target/arm/sve_helper.c | 41 ++++++++++++++++++++++++++++++++++-------
> target/arm/translate-sve.c | 38 ++++++++++++++++++++++++++++++++++++++
> target/arm/sve.decode | 7 +++++++
> 4 files changed, 93 insertions(+), 7 deletions(-)
> +/* Swap 16-bit words within a 32-bit word. */
> +static inline uint32_t hswap32(uint32_t h)
> +{
> + return rol32(h, 16);
> +}
> +
> +/* Swap 16-bit words within a 64-bit word. */
> +static inline uint64_t hswap64(uint64_t h)
> +{
> + uint64_t m = 0x0000ffff0000ffffull;
> + h = rol64(h, 32);
> + return ((h & m) << 16) | ((h >> 16) & m);
> +}
> +
> +/* Swap 32-bit words within a 64-bit word. */
> +static inline uint64_t wswap64(uint64_t h)
> +{
> + return rol64(h, 32);
> +}
> +
Were there cases in earlier patches that could have used these? I forget.
I guess they're not useful enough to be worth putting in bswap.h.
> @@ -1577,13 +1611,6 @@ void HELPER(sve_rev_b)(void *vd, void *vn, uint32_t
> desc)
> }
> }
>
> -static inline uint64_t hswap64(uint64_t h)
> -{
> - uint64_t m = 0x0000ffff0000ffffull;
> - h = rol64(h, 32);
> - return ((h & m) << 16) | ((h >> 16) & m);
> -}
> -
Better to put the function in the right place to start with.
Otherwise
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- [Qemu-arm] [PATCH v2 30/67] target/arm: Implement SVE compress active elements, (continued)
- [Qemu-arm] [PATCH v2 30/67] target/arm: Implement SVE compress active elements, Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 29/67] target/arm: Implement SVE Permute - Interleaving Group, Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 32/67] target/arm: Implement SVE copy to vector (predicated), Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 31/67] target/arm: Implement SVE conditionally broadcast/extract element, Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 33/67] target/arm: Implement SVE reverse within elements, Richard Henderson, 2018/02/17
- Re: [Qemu-arm] [PATCH v2 33/67] target/arm: Implement SVE reverse within elements,
Peter Maydell <=
- [Qemu-arm] [PATCH v2 35/67] target/arm: Implement SVE Select Vectors Group, Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 34/67] target/arm: Implement SVE vector splice (predicated), Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 37/67] target/arm: Implement SVE Integer Compare - Immediate Group, Richard Henderson, 2018/02/17
- [Qemu-arm] [PATCH v2 36/67] target/arm: Implement SVE Integer Compare - Vectors Group, Richard Henderson, 2018/02/17