[Lldb-commits] [PATCH] D56237: Implement GetFileLoadAddress for the Windows process plugin
Aaron Smith via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jan 14 22:25:47 PST 2019
asmith marked an inline comment as done.
asmith added a comment.
@clayborg
================
Comment at: source/Plugins/Process/Windows/Common/ProcessWindows.cpp:858
+Status ProcessWindows::GetFileLoadAddress(const FileSpec &file, bool &is_loaded,
+ lldb::addr_t &load_addr) {
----------------
clayborg wrote:
> This entire function is doing the job of the dynamic loader and should be moved into the dynamic loader.
when do you think the changes you mentioned for the loader and lldb-server.exe would be available for review? should we abandon this and wait for those?
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56237/new/
https://reviews.llvm.org/D56237
More information about the lldb-commits
mailing list