jgorbe added inline comments.
================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFUnit.cpp:808-809
+ if (cu_index && (header.m_unit_type == llvm::dwarf::DW_UT_compile ||
+ header.m_unit_type == llvm::dwarf::DW_UT_split_compile)) {
+ header.m_index_entry = cu_index->getFromOffset(header.m_offset);
----------------
labath wrote:
> I guess this could be `header.IsTypeUnit()` (and `!header.IsTypeUnit())`)...
But `!header.IsTypeUnit` would also treat DW_UT_partial and DW_UT_skeleton as
compile units, right?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96194/new/
https://reviews.llvm.org/D96194
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
