[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH] disas/libvixl: Really suppress gcc 4.6.3 sign-com
From: |
Alex Bennée |
Subject: |
Re: [Qemu-arm] [PATCH] disas/libvixl: Really suppress gcc 4.6.3 sign-compare warnings |
Date: |
Thu, 14 Jan 2016 15:21:49 +0000 |
User-agent: |
mu4e 0.9.15; emacs 25.0.50.1 |
Peter Maydell <address@hidden> writes:
> Commit 8acc216b956 attempted to silence some sign-compare
> warnings in libvixl by adding -Wno-sign-compare to the CFLAGS
> for the relevant objects. Unfortunately it was ineffective
> because it was placed before $(QEMU_CFLAGS), so the -Wall in
> the general flags overrode -Wno-sign-compare rather than
> vice-versa. Reorder the flags so the warning suppression works.
>
> Thanks to Franz-Josef Haider <address@hidden>
> for pointing out what was wrong with the original patch.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> Alex, is there a way to test this patch against Travis before
> we commit it to master?
Mea culpa:
Reviewed-by: Alex Bennée <address@hidden>
and a conditional:
Tested-by: Alex Bennée <address@hidden>
when:
https://travis-ci.org/stsquad/qemu/builds/102371704
completes.
>
> disas/libvixl/Makefile.objs | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/disas/libvixl/Makefile.objs b/disas/libvixl/Makefile.objs
> index d1e801a..bbe7695 100644
> --- a/disas/libvixl/Makefile.objs
> +++ b/disas/libvixl/Makefile.objs
> @@ -6,6 +6,6 @@ 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 -Wno-sign-compare $(QEMU_CFLAGS)
> +$(addprefix $(obj)/,$(libvixl_OBJS)): QEMU_CFLAGS :=
> -I$(SRC_PATH)/disas/libvixl $(QEMU_CFLAGS) -Wno-sign-compare
>
> common-obj-$(CONFIG_ARM_A64_DIS) += $(libvixl_OBJS)
--
Alex Bennée