diff options
| author | Olaf Hering <olh@suse.de> | 2005-07-21 21:02:09 +0200 |
|---|---|---|
| committer | Sam Ravnborg <sam@mars.(none)> | 2005-07-21 21:45:47 +0000 |
| commit | 3c521e06fad4b4b7fe4811fb8363d12cf49f40a2 (patch) | |
| tree | 5904f3b98801b2c86f15abea327e4a9c5dbeffee | |
| parent | ce454d4d7278b815dcee957653ce388146484f5f (diff) | |
| download | lwn-3c521e06fad4b4b7fe4811fb8363d12cf49f40a2.tar.gz lwn-3c521e06fad4b4b7fe4811fb8363d12cf49f40a2.zip | |
[PATCH] kbuild: add -Wundef to global CFLAGS
A recent change to the aic scsi driver removed two defines to detect
endianness. cpp handles undefined strings as 0. As a result, the test turned
into #if 0 == 0 and the wrong code was selected.
Adding -Wundef to global CFLAGS will catch such errors.
Signed-off-by: Olaf Hering <olh@suse.de>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
| -rw-r--r-- | Makefile | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -203,7 +203,7 @@ CONFIG_SHELL := $(shell if [ -x "$$BASH" ]; then echo $$BASH; \ HOSTCC = gcc HOSTCXX = g++ -HOSTCFLAGS = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer +HOSTCFLAGS = -Wall -Wundef -Wstrict-prototypes -O2 -fomit-frame-pointer HOSTCXXFLAGS = -O2 # Decide whether to build built-in, modular, or both. @@ -348,7 +348,7 @@ LINUXINCLUDE := -Iinclude \ CPPFLAGS := -D__KERNEL__ $(LINUXINCLUDE) -CFLAGS := -Wall -Wstrict-prototypes -Wno-trigraphs \ +CFLAGS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \ -fno-strict-aliasing -fno-common \ -ffreestanding AFLAGS := -D__ASSEMBLY__ |