diff options
| author | Julius Werner <jwerner@chromium.org> | 2022-05-27 19:07:14 -0700 |
|---|---|---|
| committer | Hung-Te Lin <hungte@chromium.org> | 2022-06-07 12:34:35 +0000 |
| commit | 23d24657209d810d39a16c561ea1f68f98f8237b (patch) | |
| tree | 83b65b131687a2859d7d49aecedc202583876c50 /payloads/libpayload/tests | |
| parent | 6876f49b7e3e8e750dc40ddbe1c37b6ab12c0153 (diff) | |
| download | coreboot-23d24657209d810d39a16c561ea1f68f98f8237b.tar.gz coreboot-23d24657209d810d39a16c561ea1f68f98f8237b.tar.bz2 coreboot-23d24657209d810d39a16c561ea1f68f98f8237b.zip | |
commonlib: Clean up compiler.h
This patch contains several minor cleanups related to compiler.h:
- Replace __always_unused() (which is a Linux-specific concept that
doesn't make sense without also having __maybe_unused(), and had zero
uses in the codebase) with __unused() which moves here from helpers.h
- Add __underscores__ to the names of all attributes in the compiler
attribute shorthand macros. This is necessary to make them work in
files where the same name was already used for an identifier (e.g.
cbfstool/cbfs.h's `unused` array of file types).
- Remove libpayload's own copy of compiler.h and make it directly pull
in the commonlib/bsd copy.
Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I9644da594bb69133843c6b7f12ce50b2e45fd24b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64737
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Yu-Ping Wu <yupingso@google.com>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
Diffstat (limited to 'payloads/libpayload/tests')
| -rw-r--r-- | payloads/libpayload/tests/Makefile.inc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/payloads/libpayload/tests/Makefile.inc b/payloads/libpayload/tests/Makefile.inc index 529524ca3253..2a067e0f9f98 100644 --- a/payloads/libpayload/tests/Makefile.inc +++ b/payloads/libpayload/tests/Makefile.inc @@ -30,7 +30,8 @@ TEST_CONFIG_ := CONFIG_LP_ # Default includes -TEST_CFLAGS := -include include/kconfig.h -include include/compiler.h +TEST_CFLAGS := -include include/kconfig.h +TEST_CFLAGS += -include $(coreboottop)/src/commonlib/bsd/include/commonlib/bsd/compiler.h TEST_CFLAGS += -Iinclude -Iinclude/mock TEST_CFLAGS += -I$(coreboottop)/src/commonlib/bsd/include TEST_CFLAGS += -I$(dir $(TEST_KCONFIG_AUTOHEADER)) |