HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39409 )
Change subject: Makefile.inc: Add -Wunused-macros warning option ......................................................................
Makefile.inc: Add -Wunused-macros warning option
Change-Id: I189e799bef353382643721cdb992f351866290d0 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr --- M Makefile.inc 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/39409/1
diff --git a/Makefile.inc b/Makefile.inc index 43b29c7..ac6cf3d 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -421,7 +421,7 @@ CFLAGS_common += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes CFLAGS_common += -Wwrite-strings -Wredundant-decls -Wno-trigraphs -Wimplicit-fallthrough CFLAGS_common += -Wstrict-aliasing -Wshadow -Wdate-time -Wtype-limits -Wvla -CFLAGS_common += -Wlogical-op -Wduplicated-cond -Wdangling-else +CFLAGS_common += -Wlogical-op -Wduplicated-cond -Wdangling-else -Wunused-macros CFLAGS_common += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer CFLAGS_common += -ffunction-sections -fdata-sections -fno-pie ifeq ($(CONFIG_COMPILER_GCC),y)
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39409 )
Change subject: Makefile.inc: Add -Wunused-macros warning option ......................................................................
Patch Set 3:
I guess this will explode when hitting the several hundred MCHBAR register definitions I am adding for many platforms.
HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/39409 )
Change subject: Makefile.inc: Add -Wunused-macros warning option ......................................................................
Abandoned
-
Angel Pons (Code Review) -
HAOUAS Elyes (Code Review)