labath added a comment.

Thanks for the review. I'm not entirely proud of how I implemented this, but I 
hope it's not too ugly either. The reason I opted for this approach is that I 
needed to implement the "if the module is null then decrement the address and 
recompute" logic in two places. RegisterContextLLDB (computes the backtrace) 
and StackFrame (computes the symbol name to display). This avoids the 
duplication, but adds another argument to a bunch of functions. I don't really 
have an idea on how to make this better, but maybe you will think of something.


https://reviews.llvm.org/D32022



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

Reply via email to