derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

base: Use M5_UNLIKELY for conditional panic, etc., macros.

Browse Source 
panic_if and fail_if should happen at most once in any given simulation,
and warn_if, etc., should still not happen most of the time.

Change-Id: Iaa6cb03c11b86d84f51cc4738efb8f203de4201c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/34817
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Gabe Black
2020-09-17 20:46:19 -07:00
parent 2f6a770ccf
commit 89ffa84de6
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/base/logging.hh
View File

@@ -196,7 +196,7 @@ class Logger
*/
#define panic_if(cond, ...) \
do { \
if ((cond)) { \
if (M5_UNLIKELY(cond)) { \
panic("panic condition " # cond " occurred: %s", \
csprintf(__VA_ARGS__)); \
} \
@@ -218,7 +218,7 @@ class Logger
*/
#define fatal_if(cond, ...) \
do { \
if ((cond)) { \
if (M5_UNLIKELY(cond)) { \
fatal("fatal condition " # cond " occurred: %s", \
csprintf(__VA_ARGS__)); \
} \
@@ -262,13 +262,13 @@ class Logger
*/
#define warn_if(cond, ...) \
do { \
if ((cond)) \
if (M5_UNLIKELY(cond)) \
warn(__VA_ARGS__); \
} while (0)
#define warn_if_once(cond, ...) \
do { \
if ((cond)) \
if (M5_UNLIKELY(cond)) \
warn_once(__VA_ARGS__); \
} while (0)
/** @} */ // end of api_logger
@@ -291,7 +291,7 @@ class Logger
#else //!NDEBUG
#define chatty_assert(cond, ...) \
do { \
if (!(cond)) \
if (M5_UNLIKELY(!(cond))) \
panic("assert(" # cond ") failed: %s", csprintf(__VA_ARGS__)); \
} while (0)
#endif // NDEBUG