From 3332c2dc6e1149b7ea4ce93244177ea2abf13349 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Wed, 1 Dec 2021 13:44:40 -0800 Subject: [PATCH] tests: Disable -Wsource-mgr for clang Unit tests fail on clang 13.0 with a cryptic error: _timestamp_size changed binding to STB_WEAK [-Werror,-Wsource-mgr] Probably something related to the weird things we do to mock memlayout areas. Too lazy to track it down. Let's just disable this (clang-specific, not properly documented) warning flag. Signed-off-by: Julius Werner Change-Id: Ie49c9eef3c74592c068c899c6717621dbcb9f609 Reviewed-on: https://review.coreboot.org/c/coreboot/+/59814 Tested-by: build bot (Jenkins) Reviewed-by: Jakub Czapiga --- payloads/libpayload/tests/Makefile.inc | 1 + tests/Makefile.inc | 1 + 2 files changed, 2 insertions(+) diff --git a/payloads/libpayload/tests/Makefile.inc b/payloads/libpayload/tests/Makefile.inc index bf484bb2c4..b59aee6eb6 100644 --- a/payloads/libpayload/tests/Makefile.inc +++ b/payloads/libpayload/tests/Makefile.inc @@ -43,6 +43,7 @@ TEST_CFLAGS += -I$(cmockasrc)/include TEST_CFLAGS += -Wall -Wundef -Wstrict-prototypes -Wvla TEST_CFLAGS += -Wwrite-strings -Wno-trigraphs -Wimplicit-fallthrough TEST_CFLAGS += -Wstrict-aliasing -Wshadow -Werror +TEST_CFLAGS += -Wno-unknown-warning-option -Wno-source-mgr TEST_CFLAGS += -std=gnu11 -Os -ffunction-sections -fdata-sections -fno-builtin diff --git a/tests/Makefile.inc b/tests/Makefile.inc index cbaf2c260a..d6258da27a 100644 --- a/tests/Makefile.inc +++ b/tests/Makefile.inc @@ -45,6 +45,7 @@ TEST_CFLAGS += -I$(src) -I$(src)/include -I$(src)/commonlib/include \ # Only put conservative warnings here that really detect code that's obviously # unintentional. TEST_CFLAGS += -Wall -Werror -Wundef -Wstrict-prototypes -Wno-inline-asm +TEST_CFLAGS += -Wno-unknown-warning-option -Wno-source-mgr # Path for Kconfig autoheader TEST_CFLAGS += -I$(dir $(TEST_KCONFIG_AUTOHEADER))