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__ |
