For those kinds of cases, we could use obj2yaml and check in yaml right?  Fwiw I tried to round-trip an exe through obj->yaml->obj recently and the resulting exe was incorrect but it was close, so I think there’s only some small fixes needed.<br><br>In regards to your previous response, couldn’t we have lldb generate the mini dump itself as the first step of the test?<br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 3, 2019 at 11:59 PM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
In D56293#1345790 <<a href="https://reviews.llvm.org/D56293#1345790" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56293#1345790</a>>, @zturner wrote:<br>
<br>
> I don't think we can check in an executable file, we should try to compile it on the spot.  We have 1-2 existing unit tests that check in an exe and we occasionally get reports that peoples' virus scanners flag them as trojans, even though they obviously aren't.  In any case, I've been meaning to remove those tests, so I think we should set a precedent that executable binaries are never checked in.<br>
<br>
<br>
While I agree that a checked-in exe shouldn't be needed in this (and most other) cases, I am not sure about the policy in general. For example, I can see a case for having a bunch of badly corrupted binaries (things like corrupted section headers, overlapping sections in the file; things that even yaml2obj will have trouble generating) and then a test that makes sure we do something reasonable (e.g., not crash) when opening them. These are exactly the kind of files that make paranoid virus scanners sound the alarm.<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D56293/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56293/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D56293" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56293</a><br>
<br>
<br>
<br>
</blockquote></div>