[Lldb-commits] [PATCH] D92264: [lldb] [POSIX-DYLD] Update the cached exe path after attach

Michał Górny via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 7 04:23:28 PST 2020


mgorny updated this revision to Diff 309871.
mgorny edited the summary of this revision.
mgorny added a comment.

Rewritten the test wrt comments, that is:

1. Added a breakpoint-continue to ensure that the program is past all initial work before we test it.
2. Print a global variable to test it.

I have also confirmed that reading the variable fails without the code change (because of reading wrong memory address), but also the breakpoint doesn't get set correctly.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92264/new/

https://reviews.llvm.org/D92264

Files:
  lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.cpp
  lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.h
  lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
  lldb/test/API/commands/process/attach/TestProcessAttach.py
  lldb/test/API/commands/process/attach/main.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D92264.309871.patch
Type: text/x-patch
Size: 4276 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20201207/66449d72/attachment.bin>


More information about the lldb-commits mailing list