Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44105 )
Change subject: lib/gcov: Remove assert(0) ......................................................................
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 --- M src/lib/gcov-glue.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/44105/1
diff --git a/src/lib/gcov-glue.c b/src/lib/gcov-glue.c index 101aec9..14f3e3e 100644 --- a/src/lib/gcov-glue.c +++ b/src/lib/gcov-glue.c @@ -82,7 +82,7 @@ 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