Re: [Pixman] [PATCH] test: add a check for FE_DIVBYZERO

2014-02-23 Thread Thomas Petazzoni
Dear Søren Sandmann, On Sun, 23 Feb 2014 02:46:48 +0100, Søren Sandmann wrote: > > +AC_CHECK_DECL([FE_DIVBYZERO], [], [], [[#include ]]) > > +if test x$have_fe_divbyzero = xyes; then > > + AC_DEFINE(HAVE_FEDIVBYZERO, 1, [Whether we have FE_DIVBYZERO]) > > +fi > > needs to actually set the $hav

Re: [Pixman] [PATCH] test: add a check for FE_DIVBYZERO

2014-02-22 Thread Søren Sandmann
Hi, Thanks for the patch. The check > +AC_CHECK_DECL([FE_DIVBYZERO], [], [], [[#include ]]) > +if test x$have_fe_divbyzero = xyes; then > + AC_DEFINE(HAVE_FEDIVBYZERO, 1, [Whether we have FE_DIVBYZERO]) > +fi needs to actually set the $have_fe_divbyzero variable to yes or no. Otherwise HAVE_FE

[Pixman] [PATCH] test: add a check for FE_DIVBYZERO

2014-02-22 Thread Thomas Petazzoni
Some architectures, such as Microblaze, currently do not implement FE_DIVBYZERO, even though they have and feenableexcept(). This commit adds a configure.ac check to verify whether FE_DIVBYZERO is defined or not, and if not, disables the problematic code in test/utils.c. Signed-off-by: Thomas Pet