[PATCH] D59634: Add minidump support to obj2yaml

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 2 04:19:38 PDT 2019


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





More information about the llvm-commits mailing list