<div dir="ltr">On Fri, Jun 24, 2016 at 7:22 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">compnerd added a subscriber: compnerd.<br>
compnerd accepted this revision.<br>
compnerd added a reviewer: compnerd.<br>
compnerd added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Nice!  I think it would be a good idea to add a few tests.  Basically, feed it a yaml document and ensure that the format detection happens correctly.<br></blockquote><div><br></div><div>Just to add this for the record, this should also correct an instance of an assert triggering in an Asserts built with the incorrect format being selected.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
================<br>
Comment at: tools/yaml2obj/yaml2macho.cpp:450<br>
@@ -449,3 +449,3 @@<br>
   }<br>
   if (auto Err = writeFatHeader(OS))<br>
     return Err;<br>
----------------<br>
Not your fault, but its hard to see that the code here is switching between a fat macho or a sliced/lipo'ed/thin one.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21711" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21711</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>