<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 8, 2014 at 7:25 AM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 8 May 2014 10:01, Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br>

> On Thu, May 08, 2014 at 09:54:32AM -0400, Ed Maste wrote:<br>
>> On 8 May 2014 09:25, Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br>
>> > I find this not really helpful and just obscuring the source.<br>
>><br>
>> I don't disagree, but I preferred to be consistent what's already<br>
>> there.  I'm happy enough to revert this and make the clang change<br>
>> below, if that's preferable.<br>
><br>
> Yes, please. For whatever reason that Apple decided to have multiple OS<br>
> names, it doesn't apply to FreeBSD.<br>
<br>
</div>Nor Linux, although isOSLinux() was added in r189604 and equivalent<br>
checks were changed to use it.<br></blockquote><div><br></div><div>My $0.02 USD (not sure if its worth anything though) is that we keep this.  Having had to deal with renaming the triples for Windows, having these methods makes for a much easier and cleaner interaction between LLVM and clang (and lldb as well).  It also allows for LLVM to change internals without breaking external consumers.</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">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>