[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: |
Ed Maste |
Subject: |
Re: [PATCH v2 6/8] qemu/bswap: Use compiler __builtin_bswap() on FreeBSD |
Date: |
Mon, 28 Sep 2020 17:16:40 -0400 |
On Mon, 28 Sep 2020 at 09:20, Philippe Mathieu-Daudé <philmd@redhat.com> 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>
Reviewed-by: Ed Maste <emaste@FreeBSD.org>
Aside, to test building with this change on Clang 11 (default compiler
in FreeBSD-CURRENT) I needed a few other changes to avoid warnings:
hw/s390x/ipl.h, hw/usb/dev-uas.c variable sized type warnings
../hw/usb/dev-uas.c:157:31: error: field 'status' with variable sized
type 'uas_iu' not at the end of a struct or class is a GNU extension
[-Werror,-Wgnu-variable-sized-type-not-at-end]
uas_iu status;
^
target/s390x/cpu_models.c pointer to smaller integer type cast
../target/s390x/cpu_models.c:984:21: error: cast to smaller integer
type 'S390Feat' from 'void *' [-Werror,-Wvoid-pointer-to-enum-cast]
S390Feat feat = (S390Feat) opaque;
^~~~~~~~~~~~~~~~~
[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