Mercurial > hg > octave-shane > gnulib-hg
changeset 16601:3846767a473f
roundf: Don't override undeclared system function on IRIX 6.5.
* m4/roundf.m4 (gl_FUNC_ROUNDF): Search for roundf() in the libraries
also when it is not declared. Set HAVE_ROUNDF. For replacement code,
test HAVE_ROUNDF, not HAVE_DECL_ROUNDF.
* modules/roundf (configure.ac): For replacement code, test
HAVE_ROUNDF, not HAVE_DECL_ROUNDF.
(Depends-on): Update conditions.
* modules/roundf-ieee (Depends-on): Update conditions.
* doc/posix-functions/roundf.texi: Mention the IRIX problem.
author | Bruno Haible <bruno@clisp.org> |
---|---|
date | Thu, 08 Mar 2012 01:55:40 +0100 |
parents | fce13a4b73aa |
children | 549120e12a45 |
files | ChangeLog doc/posix-functions/roundf.texi m4/roundf.m4 modules/roundf modules/roundf-ieee |
diffstat | 5 files changed, 65 insertions(+), 34 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +2012-03-07 Bruno Haible <bruno@clisp.org> + + roundf: Don't override undeclared system function on IRIX 6.5. + * m4/roundf.m4 (gl_FUNC_ROUNDF): Search for roundf() in the libraries + also when it is not declared. Set HAVE_ROUNDF. For replacement code, + test HAVE_ROUNDF, not HAVE_DECL_ROUNDF. + * modules/roundf (configure.ac): For replacement code, test + HAVE_ROUNDF, not HAVE_DECL_ROUNDF. + (Depends-on): Update conditions. + * modules/roundf-ieee (Depends-on): Update conditions. + * doc/posix-functions/roundf.texi: Mention the IRIX problem. + 2012-03-07 Bruno Haible <bruno@clisp.org> round: Don't override undeclared system function on IRIX 6.5.
--- a/doc/posix-functions/roundf.texi +++ b/doc/posix-functions/roundf.texi @@ -10,10 +10,10 @@ @itemize @item This function is missing on some platforms: -FreeBSD 5.2.1, OpenBSD 3.8, Minix 3.1.8, AIX 5.1, HP-UX 11, IRIX 6.5, OSF/1 4.0, Solaris 9, MSVC 9, Interix 3.5. +FreeBSD 5.2.1, OpenBSD 3.8, Minix 3.1.8, AIX 5.1, HP-UX 11, older IRIX 6.5, OSF/1 4.0, Solaris 9, MSVC 9, Interix 3.5. @item This function is not declared on some platforms: -glibc 2.8, OSF/1 5.1. +glibc 2.8, IRIX 6.5, OSF/1 5.1. @item This functions returns a wrong result for x = 1/2 - 2^-25 on some platforms: mingw.
--- a/m4/roundf.m4 +++ b/m4/roundf.m4 @@ -1,4 +1,4 @@ -# roundf.m4 serial 15 +# roundf.m4 serial 16 dnl Copyright (C) 2007-2012 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -8,23 +8,39 @@ [ m4_divert_text([DEFAULTS], [gl_roundf_required=plain]) AC_REQUIRE([gl_MATH_H_DEFAULTS]) + dnl Persuade glibc <math.h> to declare roundf(). AC_REQUIRE([gl_USE_SYSTEM_EXTENSIONS]) - AC_CHECK_DECLS([roundf], , , [[#include <math.h>]]) - if test "$ac_cv_have_decl_roundf" = yes; then - gl_CHECK_MATH_LIB([ROUNDF_LIBM], [x = roundf (x);]) - if test "$ROUNDF_LIBM" != missing; then - dnl Test whether roundf() produces correct results. On mingw, for - dnl x = 1/2 - 2^-25, the system's roundf() returns a wrong result. - AC_REQUIRE([AC_PROG_CC]) - AC_REQUIRE([AC_CANONICAL_HOST]) dnl for cross-compiles - AC_CACHE_CHECK([whether roundf works], [gl_cv_func_roundf_works], - [ - save_LIBS="$LIBS" - LIBS="$LIBS $ROUNDF_LIBM" - AC_RUN_IFELSE([AC_LANG_SOURCE([[ + + gl_CHECK_MATH_LIB([ROUNDF_LIBM], [x = roundf (x);], + [extern + #ifdef __cplusplus + "C" + #endif + float roundf (float); + ]) + if test "$ROUNDF_LIBM" != missing; then + HAVE_ROUNDF=1 + dnl Also check whether it's declared. + dnl IRIX 6.5 has roundf() in libm but doesn't declare it in <math.h>. + AC_CHECK_DECLS([roundf], , [HAVE_DECL_ROUNDF=0], [[#include <math.h>]]) + + dnl Test whether roundf() produces correct results. On mingw, for + dnl x = 1/2 - 2^-25, the system's roundf() returns a wrong result. + AC_REQUIRE([AC_PROG_CC]) + AC_REQUIRE([AC_CANONICAL_HOST]) dnl for cross-compiles + AC_CACHE_CHECK([whether roundf works], [gl_cv_func_roundf_works], + [ + save_LIBS="$LIBS" + LIBS="$LIBS $ROUNDF_LIBM" + AC_RUN_IFELSE([AC_LANG_SOURCE([[ #include <float.h> #include <math.h> +extern +#ifdef __cplusplus +"C" +#endif +float roundf (float); #ifdef _MSC_VER # pragma fenv_access (off) #endif @@ -40,20 +56,17 @@ volatile float x = 0.5f - 0.5f / TWO_MANT_DIG; exit (x < 0.5f && roundf (x) != 0.0f); }]])], [gl_cv_func_roundf_works=yes], [gl_cv_func_roundf_works=no], - [case "$host_os" in - mingw*) gl_cv_func_roundf_works="guessing no";; - *) gl_cv_func_roundf_works="guessing yes";; - esac - ]) - LIBS="$save_LIBS" + [case "$host_os" in + mingw*) gl_cv_func_roundf_works="guessing no";; + *) gl_cv_func_roundf_works="guessing yes";; + esac ]) - case "$gl_cv_func_roundf_works" in - *no) ROUNDF_LIBM=missing ;; - esac - fi - if test "$ROUNDF_LIBM" = missing; then - REPLACE_ROUNDF=1 - fi + LIBS="$save_LIBS" + ]) + case "$gl_cv_func_roundf_works" in + *no) REPLACE_ROUNDF=1 ;; + esac + m4_ifdef([gl_FUNC_ROUNDF_IEEE], [ if test $gl_roundf_required = ieee && test $REPLACE_ROUNDF = 0; then AC_CACHE_CHECK([whether roundf works according to ISO C 99 with IEC 60559], @@ -67,6 +80,11 @@ # define __NO_MATH_INLINES 1 /* for glibc */ #endif #include <math.h> +extern +#ifdef __cplusplus +"C" +#endif +float roundf (float); ]gl_FLOAT_MINUS_ZERO_CODE[ ]gl_FLOAT_SIGNBIT_CODE[ static float dummy (float f) { return 0; } @@ -95,9 +113,10 @@ fi ]) else + HAVE_ROUNDF=0 HAVE_DECL_ROUNDF=0 fi - if test $HAVE_DECL_ROUNDF = 0 || test $REPLACE_ROUNDF = 1; then + if test $HAVE_ROUNDF = 0 || test $REPLACE_ROUNDF = 1; then dnl Find libraries needed to link lib/roundf.c. AC_CHECK_DECLS([ceilf, floorf], , , [[#include <math.h>]]) if test "$ac_cv_have_decl_floorf" = yes \
--- a/modules/roundf +++ b/modules/roundf @@ -12,11 +12,11 @@ Depends-on: math extensions -float [test $HAVE_DECL_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] +float [test $HAVE_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] configure.ac: gl_FUNC_ROUNDF -if test $HAVE_DECL_ROUNDF = 0 || test $REPLACE_ROUNDF = 1; then +if test $HAVE_ROUNDF = 0 || test $REPLACE_ROUNDF = 1; then AC_LIBOBJ([roundf]) fi gl_MATH_MODULE_INDICATOR([roundf])
--- a/modules/roundf-ieee +++ b/modules/roundf-ieee @@ -8,8 +8,8 @@ Depends-on: roundf -floorf-ieee [test $HAVE_DECL_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] -ceilf-ieee [test $HAVE_DECL_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] +floorf-ieee [test $HAVE_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] +ceilf-ieee [test $HAVE_ROUNDF = 0 || test $REPLACE_ROUNDF = 1] configure.ac: gl_FUNC_ROUNDF_IEEE