[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

[Rafael Ávila De Espíndola]

 __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