qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] configure: Disable (clang) initializer-override


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.html

In 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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]