Re: [PATCH v3 0/6] qemu/bswap: Use compiler __builtin_bswap()

[Thomas Huth]

On 13/01/2023 08.05, Philippe Mathieu-Daudé wrote:
> On 13/1/23 02:05, Richard Henderson wrote:
> > 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.
>
> OK I'll double-check.

Please note that "make vm-build-haiku.x86_64" is currently broken - we need 
an update to Beta 4 from Alexander first.

 Thomas