clayborg added a comment.

In D55522#1325839 <https://reviews.llvm.org/D55522#1325839>, @labath wrote:

> This patch certainly provides a more comprehensive treatment of memory region 
> information in minidump. However, there might be some value in removing the 
> excessive shared_ptrs in the other patch. I think that should be evaluated 
> separately.


Agreed. I will make that patch following this one.


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

https://reviews.llvm.org/D55522



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

Reply via email to