diff options
author | Olaf Hering <olh@suse.de> | 2005-07-21 15:02:09 -0400 |
---|---|---|
committer | Sam Ravnborg <sam@mars.(none)> | 2005-07-21 17:45:47 -0400 |
commit | 3c521e06fad4b4b7fe4811fb8363d12cf49f40a2 (patch) | |
tree | 5904f3b98801b2c86f15abea327e4a9c5dbeffee | |
parent | ce454d4d7278b815dcee957653ce388146484f5f (diff) |
[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; \ | |||
203 | 203 | ||
204 | HOSTCC = gcc | 204 | HOSTCC = gcc |
205 | HOSTCXX = g++ | 205 | HOSTCXX = g++ |
206 | HOSTCFLAGS = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer | 206 | HOSTCFLAGS = -Wall -Wundef -Wstrict-prototypes -O2 -fomit-frame-pointer |
207 | HOSTCXXFLAGS = -O2 | 207 | HOSTCXXFLAGS = -O2 |
208 | 208 | ||
209 | # Decide whether to build built-in, modular, or both. | 209 | # Decide whether to build built-in, modular, or both. |
@@ -348,7 +348,7 @@ LINUXINCLUDE := -Iinclude \ | |||
348 | 348 | ||
349 | CPPFLAGS := -D__KERNEL__ $(LINUXINCLUDE) | 349 | CPPFLAGS := -D__KERNEL__ $(LINUXINCLUDE) |
350 | 350 | ||
351 | CFLAGS := -Wall -Wstrict-prototypes -Wno-trigraphs \ | 351 | CFLAGS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \ |
352 | -fno-strict-aliasing -fno-common \ | 352 | -fno-strict-aliasing -fno-common \ |
353 | -ffreestanding | 353 | -ffreestanding |
354 | AFLAGS := -D__ASSEMBLY__ | 354 | AFLAGS := -D__ASSEMBLY__ |