[Lldb-commits] [PATCH] D61732: FuncUnwinders: Add a new "SymbolFile" unwind plan

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri May 10 00:42:44 PDT 2019

labath added a comment.

In D61732#1497609 <https://reviews.llvm.org/D61732#1497609>, @jasonmolenda wrote:

> I'm not super concerned about the size of the UnwindPlan or FuncUnwinders objects - we create them lazily as we unwind through functions in the process, so my guess is that even a long-running debug session will have on the order of thousands of these objects.  The ABIs don't even bother to issue a single ArchDefault unwind plan and FunctionEntry unwind plan, handing out a new one for each FuncUnwinders object (ok that part is a little embarrassing, but again this isn't super high on the priority list to fix imo.)

Ok, sounds good. I'll whip up a patch to merge the _sp and _tried fields into one entity.



More information about the lldb-commits mailing list