sgraenitz added inline comments.
================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFUnit.cpp:468
// situation. Check the line tables and build the arange table from this.
SymbolContext sc;
sc.comp_unit = dwarf->GetCompUnitForDWARFCompUnit(this);
----------------
@clayborg Any idea how to reproduce this code path? I expected test_lt as below
would do, but debug_aranges is read from DIE even there. Is it (only) for
backwards compatibility?
```
clang -c -gline-tables-only -o main_lt.o main.c
clang -c -gline-tables-only -o cu1_lt.o cu1.c
clang -gline-tables-only main_lt.o cu1_lt.o -o test_lt
```
================
Comment at: source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp:1531
+ if (m_compile_unit_infos.size() > 1)
+ return 0;
+
----------------
vsk wrote:
> sgraenitz wrote:
> > sgraenitz wrote:
> > > Skipping AddOSOARanges() here.
> > > Could you leave an in-source comment explaining why this is commented out?
> >
> > @vsk AddOSOARanges() was commented out only temporarily. I reverted it in
> > the latest patch and added these lines with a comment. Is that what you
> > were asking for?
> Ah sure sure, sorry, I missed the broader context of the change.
No worries, thanks for having a look.
https://reviews.llvm.org/D52375
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
