[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 6/8] qemu/bswap: Use compiler __builtin_bswap() on FreeBSD
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 6/8] qemu/bswap: Use compiler __builtin_bswap() on FreeBSD |
Date: |
Mon, 28 Sep 2020 19:58:38 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/28/20 7:57 PM, Philippe Mathieu-Daudé wrote:
> On 9/28/20 7:32 PM, Thomas Huth wrote:
>> On 28/09/2020 15.19, Philippe Mathieu-Daudé wrote:
>>> Since commit efc6c070aca ("configure: Add a test for the minimum
>>> compiler version") the minimum compiler version required for GCC
>>> is 4.8, which supports __builtin_bswap().
>>> Remove the FreeBSD specific ifdef'ry.
>>>
>>> This reverts commit de03c3164accc21311c39327601fcdd95da301f3
>>> ("bswap: Fix build on FreeBSD 10.0").
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>> ---
>>> Cc: Ed Maste <emaste@freebsd.org>
>>> ---
>>> include/qemu/bswap.h | 2 --
>>> 1 file changed, 2 deletions(-)
>>>
>>> diff --git a/include/qemu/bswap.h b/include/qemu/bswap.h
>>> index 719d620bfe6..1a297bfec22 100644
>>> --- a/include/qemu/bswap.h
>>> +++ b/include/qemu/bswap.h
>>> @@ -6,8 +6,6 @@
>>> #ifdef CONFIG_MACHINE_BSWAP_H
>>> # include <sys/endian.h>
>>> # include <machine/bswap.h>
>>> -#elif defined(__FreeBSD__)
>>> -# include <sys/endian.h>
>>
>> Ah, well, no I get it ... you're removing this stuff bit by bit. Quite
>> confusing, IMHO, I'd remove them all in one patch instead.
Actually I thought it would be easier for each OS maintainer to
review the corresponding patch.
>
> Sure. I did it that way because I was testing OS after OS, when
> one was successful I committed the change and went for the next
> one.
>
>>
>> Thomas
>>
>
[PATCH v2 7/8] qemu/bswap: Use compiler __builtin_bswap() on NetBSD, Philippe Mathieu-Daudé, 2020/09/28
[RFC PATCH v2 8/8] tests/vm: Add Haiku test based on their vagrant images, Philippe Mathieu-Daudé, 2020/09/28