[Lldb-commits] [PATCH] [ProcessWindows] Improve support for launching processes.

Zachary Turner zturner at google.com
Tue Nov 11 16:04:06 PST 2014


Hi xen2,

Because I structured the core debug loop differently from your original patch, I'm mostly re-writing your patch rather than doing a straight merge, although I'm heavily referring to your patch in the process.  I do plan to credit your original patch when I commit though.  Would appreciate some feedback since you've already gone through all this learning before.  I'm doing this in small pieces rather than all at once, so there's still many pieces of your patch which are not here yet.

This particular piece sends notifications for module load / unload to the process plugin, and also fixes some issues managing the state more accurately during the loading sequence.

Not yet implemented: any kind of thread management, dynamic loader plugin, attaching to processes, breakpoint management other than getting the low level exception, or stepping.

http://reviews.llvm.org/D6224

Files:
  source/Plugins/Process/Windows/DebuggerThread.cpp
  source/Plugins/Process/Windows/DebuggerThread.h
  source/Plugins/Process/Windows/IDebugDelegate.h
  source/Plugins/Process/Windows/LocalDebugDelegate.cpp
  source/Plugins/Process/Windows/LocalDebugDelegate.h
  source/Plugins/Process/Windows/ProcessWindows.cpp
  source/Plugins/Process/Windows/ProcessWindows.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6224.16067.patch
Type: text/x-patch
Size: 15223 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141112/097cb486/attachment.bin>


More information about the lldb-commits mailing list