[coreboot-gerrit] New patch to review for coreboot: d30fca8 src/Kconfig: Don't treat warns as errors on Clang builds yet

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Tue Oct 28 20:18:03 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/7230

-gerrit

commit d30fca881f3589bf518f04db6f1666bc4a9b8578
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Wed Oct 29 06:00:15 2014 +1100

    src/Kconfig: Don't treat warns as errors on Clang builds yet
    
    Currently clang/llvm builds are not fully supported. As such,
    let us tone down treating errors as warnings until we actually
    build the entire tree.
    
    Change-Id: If7f90f9887e27250f5e6d73d1692505676be9d47
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/Kconfig | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/Kconfig b/src/Kconfig
index 5255130..4deed2a 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1116,9 +1116,13 @@ config ENABLE_APIC_EXT_ID
 	bool
 	default n
 
+# XXX Currently clang/llvm builds are not fully supported,
+# Let us tone down warns treated as errors until we actually build
+# the whole treee.
 config WARNINGS_ARE_ERRORS
 	bool
-	default y
+	default y if !COMPILER_LLVM_CLANG
+	default n if COMPILER_LLVM_CLANG
 
 # The four POWER_BUTTON_DEFAULT_ENABLE, POWER_BUTTON_DEFAULT_DISABLE,
 # POWER_BUTTON_FORCE_ENABLE and POWER_BUTTON_FORCE_DISABLE options are



More information about the coreboot-gerrit mailing list