<div dir="ltr">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><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">Just noticed I forgot llvm-commits. Sorry about that.<br>
<div class="HOEnZb"><div class="h5"><br>
On 11 August 2015 at 10:52, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">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>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><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>