diff options
author | Patrick Georgi <pgeorgi@google.com> | 2020-08-03 08:40:11 +0200 |
---|---|---|
committer | Angel Pons <th3fanbus@gmail.com> | 2020-08-03 10:36:00 +0000 |
commit | ac01106743705b3f27901635c0ff6a45b78e0449 (patch) | |
tree | bd9685625d79577bbb21aa3fc89fd61e0e685c50 | |
parent | 4337a9acaa1fa0eaba409a75a7c9f09c2fa04d21 (diff) |
lib/gcov: Remove assert(0)
This follows CB:44047 which probably missed this because it's a
custom assert macro (in code that has only recently been added to
build checks). Without this change, building with gcov fails because
gcc_assert(0) can be build-time verified (as introduced by CB:44044)
while we need runtime failure semantics here.
Change-Id: I71a38631955a6a45abe90f2b9ce3a924cc5d6837
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44105
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
-rw-r--r-- | src/lib/gcov-glue.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/gcov-glue.c b/src/lib/gcov-glue.c index 101aec9b7f..14f3e3ec3b 100644 --- a/src/lib/gcov-glue.c +++ b/src/lib/gcov-glue.c @@ -82,7 +82,7 @@ static int fseek(FILE *stream, long offset, int whence) static long ftell(FILE *stream) { /* ftell should currently not be called */ - gcc_assert(0); + BUG(); #if CONFIG(DEBUG_COVERAGE) printk(BIOS_DEBUG, "ftell %s\n", stream->filename); #endif |