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

Michael Sartain mikesart at gmail.com
Wed Sep 11 15:07:01 PDT 2013


(Sending again without huge phabricator attachment...)

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/9fa2bb48/attachment.html>


More information about the lldb-dev mailing list