[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] qemu/bswap: Replace bswapXX() by compiler __builtin_bswa
From: |
Richard Henderson |
Subject: |
Re: [PATCH 2/4] qemu/bswap: Replace bswapXX() by compiler __builtin_bswap() |
Date: |
Thu, 24 Sep 2020 12:25:46 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 9/23/20 3:02 AM, Philippe Mathieu-Daudé wrote:
> Use the compiler built-in function to byte swap values,
> as the compiler is clever and will fold constants.
>
> Suggested-by: Richard Henderson <richard.henderson@linaro.org>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> include/qemu/bswap.h | 30 ++++--------------------------
> 1 file changed, 4 insertions(+), 26 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
- [PATCH 0/4] qemu/bswap: Use compiler __builtin_bswap(), Philippe Mathieu-Daudé, 2020/09/23
- [PATCH 1/4] qemu/bswap: Remove unused qemu_bswap_len(), Philippe Mathieu-Daudé, 2020/09/23
- [PATCH 2/4] qemu/bswap: Replace bswapXX() by compiler __builtin_bswap(), Philippe Mathieu-Daudé, 2020/09/23
- Re: [PATCH 2/4] qemu/bswap: Replace bswapXX() by compiler __builtin_bswap(),
Richard Henderson <=
- [PATCH 3/4] qemu/bswap: Replace bswapXXs() by compiler __builtin_bswap(), Philippe Mathieu-Daudé, 2020/09/23
- [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Philippe Mathieu-Daudé, 2020/09/23
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Peter Maydell, 2020/09/23
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Philippe Mathieu-Daudé, 2020/09/23
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Richard Henderson, 2020/09/24
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Peter Maydell, 2020/09/24
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Philippe Mathieu-Daudé, 2020/09/25
- Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Richard Henderson, 2020/09/25
Re: [PATCH 4/4] qemu/bswap: Remove <byteswap.h> dependency, Richard Henderson, 2020/09/24