[Lldb-commits] [lldb] r186207 - Introduces core file support for Linux x86-64 using 'lldb a.out -c core'.
gclayton at apple.com
Fri Jul 12 15:59:02 PDT 2013
I reverted the ELF core file support until we can get the darwin build working. I could have commented out the initialization in lldb.cpp, but there were changes in AuxVector that used the ELF core file process plug-in name, so the ELF core file process plug-in needed to be compiled in in order to link.
It seems like we need to cleanup the register context classes to make sure ones that are used under ProcessPOSIX are completely separate from the ones that ProcessElfCore will use. The reverted patch had the base RegisterContext class with a GetMonitor() accessor that required ProcessPOSIX to be compiled in which doesn't work for darwin builds since ProcessPOSIX is only compiled in on systems that natively use it.
I will be happy to get the Xcode project building with any additional source files once they build and link for darwin.
% svn commit
Transmitting file data ...................
Committed revision 186223.
On Jul 12, 2013, at 3:30 PM, Greg Clayton <gclayton at apple.com> wrote:
> RegisterContextCoreLinux_x86_64 inherits from RegisterContextLinux_x86_64 which inherits from RegisterContext_x86_64 which uses has:
> ProcessMonitor &GetMonitor();
> This register context used by the core file can't use this since the process plug-in will be ProcessElfCore and the implementation of GetMonitor() does:
> ProcessMonitor &
> ProcessSP base = CalculateProcess();
> ProcessPOSIX *process = static_cast<ProcessPOSIX*>(base.get());
> return process->GetMonitor();
> ProcessELFCore doesn't, nor should it inherit from ProcessPOSIX:
> class ProcessElfCore : public lldb_private::Process
More information about the lldb-commits