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
