Re: [Lldb-commits] [PATCH] D14536: Add empty symbols to symtab for skipped symbols

2015-11-12 Thread Tamas Berghammer via lldb-commits
tberghammer abandoned this revision. tberghammer added a comment. I haven't noticed we already store the symbol index. Abandon it in favor of the API you suggested http://reviews.llvm.org/D14536 ___ lldb-commits mailing list lldb-commits@lists.llvm

Re: [Lldb-commits] [PATCH] D14536: Add empty symbols to symtab for skipped symbols

2015-11-11 Thread Greg Clayton via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. Symbols are the #1 most expensive item memory wise in LLDB right now. We remove many symbols in Mach-O and we set the m_uid of each symbol to the original symbol table index. The

[Lldb-commits] [PATCH] D14536: Add empty symbols to symtab for skipped symbols

2015-11-10 Thread Tamas Berghammer via lldb-commits
tberghammer created this revision. tberghammer added a reviewer: clayborg. tberghammer added a subscriber: lldb-commits. Herald added a subscriber: aemerson. Add empty symbols to symtab for skipped symbols LLDB skips several special purpose symbols during symtab parsing (e.g.: arm/aarch64 mapping