[Mesa-dev] [PATCH] mesa: Do not include fpu_control.h

Brian Paul brianp at vmware.com
Mon Aug 15 13:58:32 PDT 2011


On 08/15/2011 02:54 PM, Eric Anholt wrote:
> On Mon, 15 Aug 2011 11:26:51 -0700, Chad Versace<chad at chad-versace.us>  wrote:
>> Remove the inclusion of fpu_control.h from compiler.h.  Since compiler.h
>> uses no symbol from fpu_control.h, the removal is safe.
>>
>> Since Bionic lacks fpu_control.h, its inclusion breaks the Android build.
>
> I think this means that the one use of fpu_control.h in debug.c will
> never trigger any more.  I suspect it's Brian's code, and if he wants to
> keep it, we should put this under an autoconf-detected
> HAVE_FPU_CONTROL_H #define.  fpu_control.h is breaking my 32-bit builds
> because the header isn't present in the cross environment apparently,
> and I've never used that debug code so I'm not really interested in
> keeping it, myself.

I haven't used that code in years so I have no problem with removing it.

-Brian



More information about the mesa-dev mailing list