From fb01c01bf3f60d1d229126ea4088519adae5c015 Mon Sep 17 00:00:00 2001 From: Lawrence D'Anna Date: Tue, 29 Oct 2019 20:07:08 -0700 Subject: [PATCH] [LLDB][Python] warning fix for LLDBSwigPythonBreakpointCallbackFunction This is a quick followup to this commit: https://reviews.llvm.org/rGa69bbe02a2352271e8b14542073f177e24c499c1 In that, I #pragma-squelch this warning in `ScriptInterpreterPython.cpp` but we get the same warning in `PythonTestSuite.cpp`. This patch squelches the same warning in the same way as the reviweed commit. I'm submitting it without review under the "obviously correct" rule. At least if this is incorrect the main commit was also incorrect. By the way, as far as I can tell, these functions are extern "C" because SWIG does that to everything, not because they particularly need to be. --- lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp b/lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp index 2ac631eba424..12ffdfe79ec3 100644 --- a/lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp +++ b/lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp @@ -59,6 +59,9 @@ extern "C" void init_lldb(void) {} #define LLDBSwigPyInit init_lldb #endif +#pragma clang diagnostic push +#pragma clang diagnostic ignored "-Wreturn-type-c-linkage" + extern "C" llvm::Expected LLDBSwigPythonBreakpointCallbackFunction( const char *python_function_name, const char *session_dictionary_name, const lldb::StackFrameSP &sb_frame, @@ -67,6 +70,8 @@ extern "C" llvm::Expected LLDBSwigPythonBreakpointCallbackFunction( return false; } +#pragma clang diagnostic pop + extern "C" bool LLDBSwigPythonWatchpointCallbackFunction( const char *python_function_name, const char *session_dictionary_name, const lldb::StackFrameSP &sb_frame, const lldb::WatchpointSP &sb_wp) {