[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
Re: [Qemu-arm] [PATCH] disas/libvixl: Really suppress gcc 4.6.3 sign-compare warnings
Thu, 14 Jan 2016 18:27:47 +0000
On 14 January 2016 at 15:21, Alex Bennée <address@hidden> wrote:
> 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>
Test travis build completed OK, so I have pushed this fix to master.