[coreboot-gerrit] [XS] Change in coreboot[main]: scan-build: Don't skip Wnull-dereference Wlogical-op Wduplicated-cond

[Elyes Haouas (Code Review)]

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/81642?usp=email )

Change subject: scan-build: Don't skip Wnull-dereference Wlogical-op Wduplicated-cond
......................................................................

scan-build: Don't skip Wnull-dereference Wlogical-op Wduplicated-cond

Change-Id: I5aa097bf1098be31ad829209a6f4aa5e050a0c19
Signed-off-by: Elyes Haouas 
---
M Makefile.mk
1 file changed, 2 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/81642/1

diff --git a/Makefile.mk b/Makefile.mk
index d7110a9..7aa92af 100644
--- a/Makefile.mk
+++ b/Makefile.mk
@@ -500,13 +500,12 @@
 CFLAGS_common += -fstrict-aliasing -ffunction-sections -fdata-sections -fno-pie
 CFLAGS_common += -Wstring-compare
 ifeq ($(CONFIG_COMPILER_GCC),y)
-CFLAGS_common += -Wold-style-declaration
+CFLAGS_common += -Wold-style-declaration -Wnull-dereference -Wlogical-op -Wduplicated-cond
 # Don't add these GCC specific flags when running scan-build
 ifeq ($(CCC_ANALYZER_OUTPUT_FORMAT),)
 CFLAGS_common += -Wno-packed-not-aligned
 CFLAGS_common += -fconserve-stack
-CFLAGS_common += -Wnull-dereference
-CFLAGS_common += -Wlogical-op -Wduplicated-cond -Wno-array-compare
+CFLAGS_common += -Wno-array-compare
 endif
 endif