[lldb] [test] Fix TestGdbRemoteThreadName code on FreeBSD
authorMichał Górny <mgorny@moritz.systems>
Thu, 5 Nov 2020 17:56:10 +0000 (18:56 +0100)
committerMichał Górny <mgorny@moritz.systems>
Thu, 5 Nov 2020 19:45:34 +0000 (20:45 +0100)
Fix TestGdbRemoteThreadName to call ::pthread_setname_np instead
of ::pthread_set_name_np on FreeBSD.  While technically both names
are correct, the former is preferable because of compatibility
with Linux.  Furthermore, the latter requires `#include <pthread_np.h>`
that was missing causing the test to fail to compile.

Differential Revision: https://reviews.llvm.org/D90862

lldb/test/API/tools/lldb-server/thread-name/main.cpp

index 898e9a3..02eea12 100644 (file)
@@ -4,9 +4,7 @@
 void set_thread_name(const char *name) {
 #if defined(__APPLE__)
   ::pthread_setname_np(name);
-#elif defined(__FreeBSD__)
-  ::pthread_set_name_np(::pthread_self(), name);
-#elif defined(__linux__)
+#elif defined(__FreeBSD__) || defined(__linux__)
   ::pthread_setname_np(::pthread_self(), name);
 #elif defined(__NetBSD__)
   ::pthread_setname_np(::pthread_self(), "%s", const_cast<char *>(name));