<div dir="ltr">On Thu, Aug 13, 2015 at 6:20 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.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">So, is everyone OK with this? I don't really have a preference as to<br>
where the enum lives, but it should live in one place and not two.</blockquote><div><br></div><div>I *think* so.  I think that moving the enum is a nice thing, but really, if its not trivial, then that can be done as a follow on change.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
On 12 August 2015 at 12:27, Dan Gohman <<a href="mailto:dan433584@gmail.com">dan433584@gmail.com</a>> wrote:<br>
> On Wed, Aug 12, 2015 at 8:37 AM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>> It does remove information from MCObjectFileInfo though. Is it really<br>
>>> desirable to pick the Triple to be the one that gets to keep this<br>
>>> information, rather than the other way around? I don't have a clear<br>
>>> view of where this is headed.<br>
>><br>
>> Even if we move it out of the triple, IMHO it is a good step to first<br>
>> have a single location.<br>
>><br>
>> And given that we support things like x86_64-pc-windows-elf in the<br>
>> command line, having this information on the Triple c++ object also<br>
>> seems reasonable to me. If we remove it from one place we should also<br>
>> remove it from the other.<br>
><br>
> I don't know what x86_64-pc-windows-elf is, or why it isn't just<br>
> parsed into just a WindowsElf OSType. I'll let the people who know<br>
> more about this topic than me make the decisions here.<br>
><br>
> Dan<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>