clayborg accepted this revision.
clayborg added a comment.

Yep! Looks good.

As a follow up patch it would be great to ask the SymbolFile classes to break 
down some memory usage. I know in the SymbolFileDWARF we have each DWARFUnit 
that may or may not parse the unit DIE or all of the DIEs. These data 
structures take up memory and we lazily try to only parse all DIEs in a 
DWARFUnit when we need to. If we index the debug info, we will load the info 
and remembers which DWARFUnits had all of their DIEs parsed, and after indexing 
we will free the memory to keep memory usage down.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117914/new/

https://reviews.llvm.org/D117914

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

Reply via email to