[Lldb-commits] [lldb] lldb: add support for thread names on Windows (PR #74731)

[Greg Clayton via lldb-commits]

================
@@ -175,3 +179,30 @@ Status TargetThreadWindows::DoResume() {
 
   return Status();
 }
+
+const char *TargetThreadWindows::GetName() {
+  Log *log = GetLog(LLDBLog::Thread);
+  HMODULE hModule = ::LoadLibraryW(L"Kernel32.dll");
+  if (hModule) {
+    auto GetThreadDescription =
+        reinterpret_cast<GetThreadDescriptionFunctionPtr>(
+            ::GetProcAddress(hModule, "GetThreadDescription"));
----------------
clayborg wrote:

This code is only compiled on windows, why do we need to get the library handle manually and get the function pointer. Can't we just `#include <processthreadsapi.h>` and then call the function directly?

https://github.com/llvm/llvm-project/pull/74731