From a0ee4c2a93425598fbbe5f664e4456000484fa89 Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Thu, 13 Apr 2023 10:59:01 -0500 Subject: [PATCH] Fix unused functions warnings in instrumentation.c. ``` clang -c -fno-strict-overflow -Wsign-compare -Wunreachable-code -DNDEBUG -g -O3 -Wall -std=c11 -Wextra -Wno-unused-parameter -Wno-missing-field-initializers -Werror=implicit-function-declaration -fvisibility=hidden -I../cpython/Include/internal -IObjects -IInclude -IPython -I. -I../cpython/Include -DPy_BUILD_CORE -o Python/instrumentation.o ../cpython/Python/instrumentation.c ../cpython/Python/instrumentation.c:129:1: warning: unused function 'monitors_equals' [-Wunused-function] monitors_equals(_Py_Monitors a, _Py_Monitors b) ^ ../cpython/Python/instrumentation.c:150:1: warning: unused function 'monitors_and' [-Wunused-function] monitors_and(_Py_Monitors a, _Py_Monitors b) ^ 2 warnings generated. ``` --- Python/instrumentation.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Python/instrumentation.c b/Python/instrumentation.c index 8dc8b01fcb0496..853e8a10e81463 100644 --- a/Python/instrumentation.c +++ b/Python/instrumentation.c @@ -125,6 +125,7 @@ is_instrumented(int opcode) { return opcode >= MIN_INSTRUMENTED_OPCODE; } +#ifndef NDEBUG static inline bool monitors_equals(_Py_Monitors a, _Py_Monitors b) { @@ -135,6 +136,7 @@ monitors_equals(_Py_Monitors a, _Py_Monitors b) } return true; } +#endif static inline _Py_Monitors monitors_sub(_Py_Monitors a, _Py_Monitors b) @@ -146,6 +148,7 @@ monitors_sub(_Py_Monitors a, _Py_Monitors b) return res; } +#ifndef NDEBUG static inline _Py_Monitors monitors_and(_Py_Monitors a, _Py_Monitors b) { @@ -155,6 +158,7 @@ monitors_and(_Py_Monitors a, _Py_Monitors b) } return res; } +#endif static inline _Py_Monitors monitors_or(_Py_Monitors a, _Py_Monitors b)