From 877d63b3d2b57ab6c66bfaf49304cde5f800c2ce Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Wed, 3 Feb 2021 22:33:30 -0800 Subject: [PATCH] base: Replace a "panic" in cprintf with an M5_UNREACHABLE. The panic was just to signal that a point in the code should be unreachable, and brought with it a thread of dependencies which would bring in more and more extra files as it was followed. Change-Id: I46fb99b91929dca78a6547bdc7635aab9a63a9f3 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40615 Tested-by: kokoro Reviewed-by: Daniel Carvalho Reviewed-by: Jason Lowe-Power Maintainer: Jason Lowe-Power --- src/base/cprintf.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/base/cprintf.cc b/src/base/cprintf.cc index 03fa3cb8f..7a3e958f2 100644 --- a/src/base/cprintf.cc +++ b/src/base/cprintf.cc @@ -33,7 +33,6 @@ #include #include "base/compiler.hh" -#include "base/logging.hh" namespace cp { @@ -239,7 +238,7 @@ Print::processFlag() break; case '%': - panic("we shouldn't get here"); + M5_UNREACHABLE; break; default: -- 2.30.2