summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2020-08-03 08:40:11 +0200
committerAngel Pons <th3fanbus@gmail.com>2020-08-03 10:36:00 +0000
commitac01106743705b3f27901635c0ff6a45b78e0449 (patch)
treebd9685625d79577bbb21aa3fc89fd61e0e685c50
parent4337a9acaa1fa0eaba409a75a7c9f09c2fa04d21 (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.c2
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