Hi folks,
will this create an issue worthy of reporting to qemu?
With regards,
Reinoud
----- Forwarded message from Richard Henderson <richard.hender...@linaro.org>
-----
Date: Thu, 12 Jan 2023 15:05:01 -1000
From: Richard Henderson <richard.hender...@linaro.org>
To: Philippe Mathieu-Daudé <phi...@linaro.org>, qemu-de...@nongnu.org
Cc: Marc-André Lureau <marcandre.lur...@redhat.com>, Ryo ONODERA
<ry...@netbsd.org>, "Daniel P. Berrangé" <berra...@redhat.com>, Thomas Huth
<th...@redhat.com>,
Ira Weiny <ira.we...@intel.com>, Paolo Bonzini <pbonz...@redhat.com>,
Kamil Rytarowski <ka...@netbsd.org>, Reinoud Zandijk <rein...@netbsd.org>,
Jonathan
Cameron <jonathan.came...@huawei.com>
Subject: Re: [PATCH v3 0/6] qemu/bswap: Use compiler __builtin_bswap()
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
Thunderbird/102.4.2
On 1/11/23 08:31, Philippe Mathieu-Daudé wrote:
> Implement Richard's suggestion to use __builtin_bswap().
>
> Convert to __builtin_bswap() one patch per OS to simplify
> maintainers review.
>
> Since v2:
> - Rebased adapting ./configure changes to meson
>
> Since v1:
> - Remove the Haiku/BSD ifdef'ry (Peter)
> - Include the Haiku VM image from Alexander
>
> Philippe Mathieu-Daudé (6):
> qemu/bswap: Replace bswapXX() by compiler __builtin_bswap()
> qemu/bswap: Replace bswapXXs() by compiler __builtin_bswap()
> qemu/bswap: Remove <byteswap.h> dependency
> qemu/bswap: Use compiler __builtin_bswap() on Haiku
> qemu/bswap: Use compiler __builtin_bswap() on FreeBSD
> qemu/bswap: Use compiler __builtin_bswap() on NetBSD
If this passes on all the odd OS's, great.
Failure on some oddball is what blocked my patch set years ago.
r~
----- End forwarded message -----
