summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2019-05-15 21:39:57 +0200
committerPatrick Georgi <pgeorgi@google.com>2019-11-18 19:19:24 +0000
commit4100c2a6e3d214dc679eb359de959de08b8d96f8 (patch)
tree71e63652d6e8e7e198798f57f8cc7a35538ae933
parent556cc26337bc3b36b20f139b5627de441eba88ef (diff)
build system: Add various compiler flags that enable warnings on UB
Some types of Undefined Behavior can be determined statically at compile time and gcc now has a set of flags that make it emit warnings in that case instead of doing the __builtin_trap() / optimize / UD2-opcode dance that silently breaks the resulting binary. BUG=chromium:958270 BRANCH=none TEST=abuild passes (probably not) Change-Id: I3aa5ca00c9838cc7517160069310a1ef85372027 Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/32814 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--Makefile.inc1
1 files changed, 1 insertions, 0 deletions
diff --git a/Makefile.inc b/Makefile.inc
index d3aa8856b4..66144cd2ac 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -419,6 +419,7 @@ CFLAGS_common += -fno-delete-null-pointer-checks
ifeq ($(CCC_ANALYZER_OUTPUT_FORMAT),)
CFLAGS_common += -Wno-packed-not-aligned
CFLAGS_common += -fconserve-stack
+CFLAGS_common += -Wnull-dereference -Wreturn-type
# cf. commit f69a99db (coreboot: x86: enable gc-sections)
CFLAGS_common += -Wno-unused-but-set-variable
endif