[llvm] r208309 - Add isOSFreeBSD triple test

Ed Maste emaste at freebsd.org
Thu May 8 07:25:26 PDT 2014


On 8 May 2014 10:01, Joerg Sonnenberger <joerg at britannica.bec.de> wrote:
> On Thu, May 08, 2014 at 09:54:32AM -0400, Ed Maste wrote:
>> On 8 May 2014 09:25, Joerg Sonnenberger <joerg at britannica.bec.de> wrote:
>> > I find this not really helpful and just obscuring the source.
>>
>> I don't disagree, but I preferred to be consistent what's already
>> there.  I'm happy enough to revert this and make the clang change
>> below, if that's preferable.
>
> Yes, please. For whatever reason that Apple decided to have multiple OS
> names, it doesn't apply to FreeBSD.

Nor Linux, although isOSLinux() was added in r189604 and equivalent
checks were changed to use it.



More information about the llvm-commits mailing list