[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)
Hui wrote:
> 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 ).



More information about the lldb-commits mailing list