diff options
| author | Martin Roth <gaumless@gmail.com> | 2018-07-22 10:11:26 -0600 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-07-24 09:08:55 +0000 |
| commit | 21e09b1c157b1b8488055bae3f973143d6f59e20 (patch) | |
| tree | f8ed7daf19514063d317bb0b79ed0fd33c27ae1d /Makefile.inc | |
| parent | fbc87b638a25819160db467f82958d2ed673b66b (diff) | |
| download | coreboot-21e09b1c157b1b8488055bae3f973143d6f59e20.tar.xz | |
Build system: Add fixes for scanbuild
- Exclude build flags that generate warnings when scanbuild is running
- Add the SCANBUILD_ARGS variable to abuild so we can pass in arguments
to scanbuild.
- Set the default scanbuild argument to -k (--keep-going) so that even
if an error occurs it continues with the scan. This is similar to what
we do with coverity runs.
Change-Id: I82e7c13d7fd7432b43c17a31834ec82fca158a07
Signed-off-by: Martin Roth <gaumless@gmail.com>
Reviewed-on: https://review.coreboot.org/27595
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'Makefile.inc')
| -rw-r--r-- | Makefile.inc | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/Makefile.inc b/Makefile.inc index 6884a90967..fe325048f8 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -385,11 +385,13 @@ CFLAGS_common += -Wstrict-aliasing -Wshadow -Wdate-time CFLAGS_common += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer CFLAGS_common += -ffunction-sections -fdata-sections -fno-pie ifeq ($(CONFIG_COMPILER_GCC),y) -CFLAGS_common += -Wno-packed-not-aligned CFLAGS_common += -fno-delete-null-pointer-checks # 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 +# cf. commit f69a99db (coreboot: x86: enable gc-sections) +CFLAGS_common += -Wno-unused-but-set-variable endif endif @@ -439,11 +441,6 @@ ADAFLAGS_common += -gnatyN LDFLAGS_common := --gc-sections -nostdlib -nostartfiles -static --emit-relocs -ifeq ($(CONFIG_COMPILER_GCC),y) -# cf. commit f69a99db (coreboot: x86: enable gc-sections) -CFLAGS_common += -Wno-unused-but-set-variable -endif - ifeq ($(CONFIG_WARNINGS_ARE_ERRORS),y) CFLAGS_common += -Werror endif |