[coreboot-gerrit] Patch set updated for coreboot: 62e2b6c Makefile: Tone down some clang warnings, some are unproductive

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Fri Dec 12 12:34:41 CET 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7778

-gerrit

commit 62e2b6c9859fa875b1ff5a828ab33f12f296cd86
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Wed Dec 10 19:30:22 2014 +1100

    Makefile: Tone down some clang warnings, some are unproductive
    
    Too many false positives, not useful enough for us at the moment
    at the cost of not having the whole tree build without warnings
    as errors.
    
    Change-Id: I9f9910b7f66ebf3a82d42e7732e413ba27dbbbe7
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 Makefile | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Makefile b/Makefile
index eaba3e4..6ef9838 100644
--- a/Makefile
+++ b/Makefile
@@ -118,6 +118,8 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
 # NOTE: clang puts compiler-rt under lib/linux/libclang_rt.builtins-i386.a
 # this means the triple is i386-linux-elf instead of i386-none-elf
 CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
+# Tone down some clang warnings
+CFLAGS_x86_32 += -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare -Wno-shift-overflow
 CC_x86_32:=clang
 HOSTCC := clang
 



More information about the coreboot-gerrit mailing list