<div dir="ltr">I'm afraid I don't see a patch, could you please re-attach one?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 11, 2015 at 10:41 PM, Saleem Abdulrasool via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Tue, Aug 11, 2015 at 1:32 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br></span><div class="gmail_extra"><span class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just noticed I forgot llvm-commits. Sorry about that.<br>
<div><div><br>
On 11 August 2015 at 10:52, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br>
> The attached patch just centralizes the knowledge of what file format<br>
> is being used in the triple.<br>
><br>
> Other than some places that were handling unknown as ELF, this should<br>
> have no change. The test updates are because we were detecting<br>
> arm-coff or x86_64-win64-coff as ELF targets before.<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></span>I really like the underlying idea here.  The only thing that Im not fully convinced by is the idea that the enumeration of various file formats we support should be part of the triple (purely a movement of the enum).  That is something that is definitely outside of the scope of this change.<br clear="all"><div><br></div><div>LGTM.</div><span class="HOEnZb"><font color="#888888"><div><br></div>-- <br><div>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</font></span></div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>