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.


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

https://reviews.llvm.org/D61732



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

Reply via email to