[Lldb-commits] [PATCH] D56237: Implement GetLoadAddress for the Windows process plugin
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Feb 4 07:36:14 PST 2019
labath added inline comments.
Comment at: source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp:82
+ // Second, try through the underlying platform.
+ load_addr = Host::GetProcessBaseAddress(m_process->GetID());
+ if (load_addr != LLDB_INVALID_ADDRESS)
> Shall not come to this point for a remote process. Need to bail out ahead.
Yes, but what's the reason for the fallback in the first place? ProcessWindows is already by definition on the host, so you could do the same work in ProcessWindows::GetFileLoadAddress (but I'm not saying you should, since that information should already be known through ProcessWindows, as it received it through the windows debug api in OnLoadDLL ).
CHANGES SINCE LAST ACTION
More information about the lldb-commits