[llvm] r183303 - The GNU/HURD is also using the libc. Therefor, endian.h should be included, not machine/endian.h. See full build log https://buildd.debian.org/status/fetch.php?pkg=llvm-toolchain-3.3&arch=hurd-i386&ver=1%3A3.3~%2Brc3-1~exp1&stamp=1370358869

[Sylvestre Ledru]

Hello Rafael,

No, __GNUC__ is defined but not __GNU__.
It is really for the GNU HURD kernel.

Sylvestre

On 10/06/2013 20:17, Rafael Ávila De Espíndola wrote:
>  __GNU__ is also defined on Linux, no? If so, please simplify the #if.
> 
> Sent from my iPhone
> 
> On 2013-06-05, at 5:17, Sylvestre Ledru <sylvestre at debian.org> wrote:
> 
>> Author: sylvestre
>> Date: Wed Jun  5 04:17:26 2013
>> New Revision: 183303
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=183303&view=rev
>> Log:
>> The GNU/HURD is also using the libc. Therefor, endian.h should be included, not machine/endian.h. See full build log https://buildd.debian.org/status/fetch.php?pkg=llvm-toolchain-3.3&arch=hurd-i386&ver=1%3A3.3~%2Brc3-1~exp1&stamp=1370358869
>>
>> Modified:
>>    llvm/trunk/include/llvm/Support/Host.h
>>
>> Modified: llvm/trunk/include/llvm/Support/Host.h
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/Host.h?rev=183303&r1=183302&r2=183303&view=diff
>> ==============================================================================
>> --- llvm/trunk/include/llvm/Support/Host.h (original)
>> +++ llvm/trunk/include/llvm/Support/Host.h Wed Jun  5 04:17:26 2013
>> @@ -16,7 +16,7 @@
>>
>> #include "llvm/ADT/StringMap.h"
>>
>> -#if defined(__linux__)
>> +#if defined(__linux__) || defined(__GNU__)
>> #include <endian.h>
>> #else
>> #if !defined(BYTE_ORDER) && !defined(LLVM_ON_WIN32)
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits