Re: [Mesa-dev] [PATCH 06/15] main: Remove problematic #include on Android

2011-08-04 Thread Eric Anholt
On Thu,  4 Aug 2011 02:47:15 -0700, Chad Versace c...@chad-versace.us wrote:
 Remove ``#include fpu_control.h`` from compiler.h.
 
 Note: This is in preparation for porting i965 to Android.
 CC: Chia-I Wu o...@lunarg.com,
 CC: Chih-Wei Huang cwhu...@android-x86.org
 Signed-off-by: Chad Versace c...@chad-versace.us
 ---
  src/mesa/main/compiler.h |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
 index d736fdf..3981644 100644
 --- a/src/mesa/main/compiler.h
 +++ b/src/mesa/main/compiler.h
 @@ -45,7 +45,7 @@
  #include stdlib.h
  #include stdio.h
  #include string.h
 -#if defined(__linux__)  defined(__i386__)
 +#if defined(__linux__)  defined(__i386__)  !defined(ANDROID)
  #include fpu_control.h
  #endif
  #include float.h
 -- 
 1.7.6

I've noticed that this include is missing on my debian cross-build
environment too.  Can we tell if it's even required on linux 386 any
more?  I doesn't appear to be to me.


pgplPSV9dWTmX.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 06/15] main: Remove problematic #include on Android

2011-08-04 Thread Chad Versace
On 08/04/2011 08:43 AM, Eric Anholt wrote:
 On Thu,  4 Aug 2011 02:47:15 -0700, Chad Versace c...@chad-versace.us wrote:
 Remove ``#include fpu_control.h`` from compiler.h.

 Note: This is in preparation for porting i965 to Android.
 CC: Chia-I Wu o...@lunarg.com,
 CC: Chih-Wei Huang cwhu...@android-x86.org
 Signed-off-by: Chad Versace c...@chad-versace.us
 ---
  src/mesa/main/compiler.h |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

 diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
 index d736fdf..3981644 100644
 --- a/src/mesa/main/compiler.h
 +++ b/src/mesa/main/compiler.h
 @@ -45,7 +45,7 @@
  #include stdlib.h
  #include stdio.h
  #include string.h
 -#if defined(__linux__)  defined(__i386__)
 +#if defined(__linux__)  defined(__i386__)  !defined(ANDROID)
  #include fpu_control.h
  #endif
  #include float.h
 -- 
 1.7.6
 
 I've noticed that this include is missing on my debian cross-build
 environment too.  Can we tell if it's even required on linux 386 any
 more?  I doesn't appear to be to me.

You're right. compiler.h uses no symbols from fpu_control.h.

-- 
Chad Versace
c...@chad-versace.us



signature.asc
Description: OpenPGP digital signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev