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

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Fri Dec 12 12:34:16 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 843f5777da49089fc1879f80963ea533ce8cbedf
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