tberghammer added a comment.

I though a bit more about the abstraction you plan to introduce (DWARFDIE) and 
I started to believe we don't need it at all. If we use lldb::user_id_t in the 
NameToDIE indexes where the first 4 byte is the compile unit offset (for dwo) 
or the compile unit index (for debug map) and the last 4 byte is the die offset 
then we will have all of the information we need.

In this setup DWARFCompileUnit::Index have to be changed to index the dwo dwarf 
files also and all function of DWARFDebugInfoEntry have to be changed to check 
if it was given the compile unit belongs to the actual DIE or it got a pointer 
to the DIE in the main object file. In the second case it re-calls itself with 
the correct SymbolFileDwarf and DWARCompileUnit objects. I think this approach 
will keep the dwo file handling in the DWARFCompileUnit and in the 
DWARFDebugInfoEntry classes.

What do you think?


http://reviews.llvm.org/D12291



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to