[Lldb-commits] [PATCH] D30981: Remove HostThreadLinux/Free/NetBSD

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 15 07:13:59 PDT 2017


labath created this revision.
Herald added a subscriber: mgorny.

These classes existed only because of the GetName() static function,
which can be moved to a more natural place anyway. I move the linux
version to NativeProcessLinux (and get rid of ProcFileReader), the
freebsd version to ProcessFreeBSD (and fix a bug where it was using the
current process ID, instead of the inferior pid), and remove the NetBSD
version (which was probably incorrect anyway, as it assumes the current
process instead of the inferior.

I also add an llgs test to that verifies thread names are read
correctly.


https://reviews.llvm.org/D30981

Files:
  include/lldb/Host/HostNativeThread.h
  include/lldb/Host/HostNativeThreadForward.h
  include/lldb/Host/freebsd/HostThreadFreeBSD.h
  include/lldb/Host/linux/HostThreadLinux.h
  include/lldb/Host/netbsd/HostThreadNetBSD.h
  packages/Python/lldbsuite/test/tools/lldb-server/thread-name/Makefile
  packages/Python/lldbsuite/test/tools/lldb-server/thread-name/TestGdbRemoteThreadName.py
  packages/Python/lldbsuite/test/tools/lldb-server/thread-name/main.cpp
  source/Host/CMakeLists.txt
  source/Host/freebsd/HostThreadFreeBSD.cpp
  source/Host/linux/HostThreadLinux.cpp
  source/Host/netbsd/HostThreadNetBSD.cpp
  source/Plugins/Process/FreeBSD/FreeBSDThread.cpp
  source/Plugins/Process/Linux/NativeThreadLinux.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30981.91869.patch
Type: text/x-patch
Size: 16072 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170315/52e9bd91/attachment-0001.bin>


More information about the lldb-commits mailing list