|
From: | Stefan Weil |
Subject: | Re: [Qemu-devel] [PATCH] configure: Disable (clang) initializer-overrides warnings |
Date: | Tue, 16 Oct 2012 22:20:25 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20120922 Iceowl/1.0b1 Icedove/3.0.11 |
Am 14.10.2012 22:24, schrieb Peter Maydell:
On 14 October 2012 21:15, Blue Swirl<address@hidden> wrote:On Sun, Oct 14, 2012 at 8:00 PM, Peter Maydell<address@hidden> wrote:Blue Swirl previously submitted a patch which enabled this flag (among others): http://lists.gnu.org/archive/html/qemu-devel/2012-07/msg04203.htmlIn my version, Clang flags were separate from GCC flags. I think it would be better to run different tests for each compiler in the future, that lets GCC avoid Clang flag -ffoobar which does not have desirable effect to GCC and vice versa.Mmm, but at the moment it's entirely fine to pass both compilers the same flags, and moreover we actually pass all these flags to the compiler without checking if it's gcc or not; we just ask if the compiler will run OK with the flag passed. (I'm not even sure how you'd determine whether your compiler was gcc or clang or gcc-llvm or whatever...)I'd also like to handle flags for sparse checker the same way.I agree that those bits of configure could be cleaner. -- PMM
Hi Peter, hi Blue,gcc uses a different flag -Wno-override-init to disable initializer override warnings.
I tested the following patch which uses -Wextra with gcc-4.4 and gcc-4.7. It allows compilation of QEMU without warnings. A test with those flags and clang still has to be done. Regards Stefan W. diff --git a/configure b/configure index 359e453..0818eda 100755 --- a/configure +++ b/configure @@ -267,6 +267,8 @@ sdl_config="${SDL_CONFIG-${cross_prefix}sdl-config}" QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS"QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS" +QEMU_CFLAGS="-Wextra -Wno-missing-field-initializers $QEMU_CFLAGS"+QEMU_CFLAGS="-Wno-override-init -Wno-sign-compare -Wno-unused-parameter $QEMU_CFLAGS" QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"
QEMU_INCLUDES="-I. -I\$(SRC_PATH) -I\$(SRC_PATH)/fpu" if test "$debug_info" = "yes"; then
[Prev in Thread] | Current Thread | [Next in Thread] |