Mailing List Archive

Add -Wdeclaration-after-statement to CFLAGS only if the
# HG changeset patch
# User kaf24@firebug.cl.cam.ac.uk
# Node ID b6ebabe4658a0c08e17853431d1c0eba13664d6e
# Parent 393256b2ead0c883e2095479e1c66d2e34d18a10
Add -Wdeclaration-after-statement to CFLAGS only if the
compiler supports it. Move test-gcc-flag function to
Config.mk where it can be used by any component of the
build.

Signed-off-by: Keir Fraser <keir@xensource.com>

diff -r 393256b2ead0 -r b6ebabe4658a Config.mk
--- a/Config.mk Tue Nov 22 18:10:50 2005
+++ b/Config.mk Wed Nov 23 10:15:18 2005
@@ -8,7 +8,6 @@
# Tools to run on system hosting the build
HOSTCC = gcc
HOSTCFLAGS = -Wall -Werror -Wstrict-prototypes -O2 -fomit-frame-pointer
-HOSTCFLAGS += -Wdeclaration-after-statement

AS = $(CROSS_COMPILE)as
LD = $(CROSS_COMPILE)ld
@@ -39,7 +38,10 @@
EXTRA_LIB += $(EXTRA_PREFIX)/$(LIBDIR)
endif

-CFLAGS += -Wdeclaration-after-statement
+test-gcc-flag = $(shell $(CC) -v --help 2>&1 | grep -q " $(1) " && echo $(1))
+
+HOSTCFLAGS += $(call test-gcc-flag,-Wdeclaration-after-statement)
+CFLAGS += $(call test-gcc-flag,-Wdeclaration-after-statement)

LDFLAGS += $(foreach i, $(EXTRA_LIB), -L$(i))
CFLAGS += $(foreach i, $(EXTRA_INCLUDES), -I$(i))
diff -r 393256b2ead0 -r b6ebabe4658a xen/Rules.mk
--- a/xen/Rules.mk Tue Nov 22 18:10:50 2005
+++ b/xen/Rules.mk Wed Nov 23 10:15:18 2005
@@ -43,8 +43,6 @@
endif
ALL_OBJS += $(BASEDIR)/arch/$(TARGET_ARCH)/arch.o

-test-gcc-flag = $(shell $(CC) -v --help 2>&1 | grep -q " $(1) " && echo $(1))
-
include $(BASEDIR)/arch/$(TARGET_ARCH)/Rules.mk

ifneq ($(debug),y)

_______________________________________________
Xen-changelog mailing list
Xen-changelog@lists.xensource.com
http://lists.xensource.com/xen-changelog