jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

In D59634#1449621 <https://reviews.llvm.org/D59634#1449621>, @labath wrote:

> James, do you have any thoughts on this? I think the minidump-specific 
> changes are all pretty straight-forward here, so the thing I'm most 
> interested in is whether this is ok from a obj2yaml perspective.


Sorry, I've been a bit snowed under with other stuff recently. I don't claim to 
be an expert on obj2yaml, but this LGTM.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59634



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

Reply via email to