diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2006-01-12 19:23:41 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2006-01-12 19:23:41 +0000 |
| commit | e2c60a2cfe3b8f12564133d18d62d6dece12c63f (patch) | |
| tree | 593754cdaf2e84b2b895dc08bbba3dc659f68702 /configure.in | |
| parent | 34b0ac6ada5e7e9ea99a4059a5ed5d5f34142147 (diff) | |
Use a more bulletproof test for whether finite() and isinf() are present.
It seems that recent gcc versions can optimize away calls to these functions
even when the functions do not exist on the platform, resulting in a bogus
positive result. Avoid this by using a non-constant argument and ensuring
that the function result is not simply discarded. Per report from
François Laupretre.
Diffstat (limited to 'configure.in')
| -rw-r--r-- | configure.in | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/configure.in b/configure.in index c57e93282ad..572f9d2265d 100644 --- a/configure.in +++ b/configure.in @@ -1,5 +1,5 @@ dnl Process this file with autoconf to produce a configure script. -dnl $PostgreSQL: pgsql/configure.in,v 1.431.2.5 2006/01/05 04:02:23 momjian Exp $ +dnl $PostgreSQL: pgsql/configure.in,v 1.431.2.6 2006/01/12 19:23:41 tgl Exp $ dnl dnl Developers, please strive to achieve this order: dnl @@ -892,12 +892,13 @@ fi AC_CHECK_DECLS([snprintf, vsnprintf]) -# do this one the hard way in case isinf() is a macro +dnl Cannot use AC_CHECK_FUNC because isinf may be a macro AC_CACHE_CHECK([for isinf], ac_cv_func_isinf, -[AC_TRY_LINK( -[#include <math.h> +[AC_TRY_LINK([ +#include <math.h> +double glob_double; ], -[double x = 0.0; int res = isinf(x);], +[return isinf(glob_double) ? 0 : 1;], [ac_cv_func_isinf=yes], [ac_cv_func_isinf=no])]) @@ -960,8 +961,11 @@ fi dnl Cannot use AC_CHECK_FUNC because finite may be a macro AC_MSG_CHECKING(for finite) -AC_TRY_LINK([#include <math.h>], - [int dummy=finite(1.0);], +AC_TRY_LINK([ +#include <math.h> +double glob_double; +], + [return finite(glob_double) ? 0 : 1;], [AC_DEFINE(HAVE_FINITE, 1, [Define to 1 if you have finite().]) AC_MSG_RESULT(yes)], [AC_MSG_RESULT(no)]) |