[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] libvixl: Correct build failures on NetBSD
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] libvixl: Correct build failures on NetBSD |
Date: |
Tue, 30 May 2017 12:03:47 +0100 |
On 14 May 2017 at 06:18, Kamil Rytarowski <address@hidden> wrote:
> Ensure that C99 macros are defined regardless of the inclusion order of
> headers in vixl. This is required at least on NetBSD.
>
> The vixl/globals.h headers defines __STDC_CONSTANT_MACROS and must be
> included before other system headers.
>
> This file defines unconditionally the following macros, without altering
> the original sources:
> - __STDC_CONSTANT_MACROS
> - __STDC_LIMIT_MACROS
> - __STDC_FORMAT_MACROS
>
> Signed-off-by: Kamil Rytarowski <address@hidden>
> ---
> disas/libvixl/Makefile.objs | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/disas/libvixl/Makefile.objs b/disas/libvixl/Makefile.objs
> index bbe7695fdb..860fb7f384 100644
> --- a/disas/libvixl/Makefile.objs
> +++ b/disas/libvixl/Makefile.objs
> @@ -7,5 +7,8 @@ libvixl_OBJS = vixl/utils.o \
> # The -Wno-sign-compare is needed only for gcc 4.6, which complains about
> # some signed-unsigned equality comparisons which later gcc versions do not.
> $(addprefix $(obj)/,$(libvixl_OBJS)): QEMU_CFLAGS :=
> -I$(SRC_PATH)/disas/libvixl $(QEMU_CFLAGS) -Wno-sign-compare
> +# Ensure that C99 macros are defined regardless of the inclusion order of
> +# headers in vixl. This is required at least on NetBSD.
> +$(addprefix $(obj)/,$(libvixl_OBJS)): QEMU_CFLAGS +=
> -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -D__STDC_FORMAT_MACROS
Thanks for this patch -- I have applied it to target-arm.next.
I've also reported the underlying libvixl bug upstream so hopefully
it will be fixed in a future libvixl version too.
thanks
-- PMM