diff options
| author | Patrick Georgi <pgeorgi@google.com> | 2019-05-15 21:39:57 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-11-18 19:19:24 +0000 |
| commit | 4100c2a6e3d214dc679eb359de959de08b8d96f8 (patch) | |
| tree | 71e63652d6e8e7e198798f57f8cc7a35538ae933 | |
| parent | 556cc26337bc3b36b20f139b5627de441eba88ef (diff) | |
| download | coreboot-4100c2a6e3d214dc679eb359de959de08b8d96f8.tar.xz | |
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.inc | 1 |
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 |