[lldb-dev] Fwd: [PATCH for REVIEW] Cleanup POSIX RegisterContext class hierarchies.

Michael Sartain mikesart at gmail.com
Wed Sep 11 15:03:34 PDT 2013


First pass at cleaning up the POSIX registercontext stuff.

Future changes:
 - We should be able to get rid of the RegisterContextPOSIX stuff for core
files (which don't need hardware breakpoints, etc.)
 - Fix up RegisterContextPOSIX_i386. I believe we can use it on all 32-bit
targets regardless of debugger platform using PTRACE_GETREGSET /
NT_PRSTATUS instead of PTRACE_GETREGS.

Thanks.
 -Mike

 (Also, thanks to Greg for suggestions and initial review...)

---------- Forwarded message ----------
From: Michael Sartain <mikesart at gmail.com>
Date: Wed, Sep 11, 2013 at 2:58 PM
Subject: [PATCH] Cleanup POSIX RegisterContext class hierarchies.
To: mikesart at gmail.com


http://llvm-reviews.chandlerc.com/D1651

Files:
  source/Plugins/Process/POSIX/CMakeLists.txt
  source/Plugins/Process/POSIX/POSIXThread.cpp
  source/Plugins/Process/POSIX/RegisterContextFreeBSD_x86_64.cpp
  source/Plugins/Process/POSIX/RegisterContextFreeBSD_x86_64.h
  source/Plugins/Process/POSIX/RegisterContextLinux_x86_64.cpp
  source/Plugins/Process/POSIX/RegisterContextLinux_x86_64.h
  source/Plugins/Process/POSIX/RegisterContextPOSIX.h
  source/Plugins/Process/POSIX/RegisterContextPOSIXProcessMonitor_i386.cpp
  source/Plugins/Process/POSIX/RegisterContextPOSIXProcessMonitor_i386.h
  source/Plugins/Process/POSIX/RegisterContextPOSIXProcessMonitor_x86_64.cpp
  source/Plugins/Process/POSIX/RegisterContextPOSIXProcessMonitor_x86_64.h
  source/Plugins/Process/POSIX/RegisterContextPOSIX_i386.cpp
  source/Plugins/Process/POSIX/RegisterContextPOSIX_i386.h
  source/Plugins/Process/POSIX/RegisterContextPOSIX_x86_64.cpp
  source/Plugins/Process/POSIX/RegisterContextPOSIX_x86_64.h
  source/Plugins/Process/POSIX/RegisterContext_i386.cpp
  source/Plugins/Process/POSIX/RegisterContext_i386.h
  source/Plugins/Process/POSIX/RegisterContext_x86_64.cpp
  source/Plugins/Process/POSIX/RegisterContext_x86_64.h
  source/Plugins/Process/elf-core/CMakeLists.txt
  source/Plugins/Process/elf-core/RegisterContextCoreFreeBSD_x86_64.cpp
  source/Plugins/Process/elf-core/RegisterContextCoreFreeBSD_x86_64.h
  source/Plugins/Process/elf-core/RegisterContextCoreLinux_x86_64.cpp
  source/Plugins/Process/elf-core/RegisterContextCoreLinux_x86_64.h
  source/Plugins/Process/elf-core/RegisterContextPOSIXCore_x86_64.cpp
  source/Plugins/Process/elf-core/RegisterContextPOSIXCore_x86_64.h
  source/Plugins/Process/elf-core/ThreadElfCore.cpp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20130911/8ea4a729/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1651.1.patch
Type: text/x-patch
Size: 159480 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20130911/8ea4a729/attachment.bin>


More information about the lldb-dev mailing list