diff options
author | Carlos O'Donell <carlos@systemhalted.org> | 2016-02-14 19:27:06 -0500 |
---|---|---|
committer | Carlos O'Donell <carlos@systemhalted.org> | 2016-02-14 20:09:52 -0500 |
commit | 3c47c83a9730c20e602694505b9278c25637b0d0 (patch) | |
tree | c6f859629e8c270ef682459d2d314e7d68ba51f1 | |
parent | 1233be76694ca81454f61e2ba5a2fb5830840191 (diff) | |
download | glibc-3c47c83a9730c20e602694505b9278c25637b0d0.zip glibc-3c47c83a9730c20e602694505b9278c25637b0d0.tar.gz glibc-3c47c83a9730c20e602694505b9278c25637b0d0.tar.bz2 |
Ensure isinff, isinfl, isnanf, and isnanl are defined (Bug 19439)
In ICO C++11 mode ensure that isinff, isinfl, isnanf, and isnanl
are defined. These functions were accidentally removed from the
header as part of commit d9b965fa56350d6eea9f7f438a0714c7ffbb183f,
but being GNU extensions, they should have been left in place.
-rw-r--r-- | ChangeLog | 15 | ||||
-rw-r--r-- | math/Makefile | 3 | ||||
-rw-r--r-- | math/bits/mathcalls.h | 8 | ||||
-rw-r--r-- | math/test-math-isinff.cc | 48 |
4 files changed, 71 insertions, 3 deletions
@@ -1,3 +1,16 @@ +2016-02-14 Jakub Jelinek <jakub@redhat.com> + Jonathan Wakely <jwakely@redhat.com> + Carlos O'Donell <carlos@redhat.com> + + [BZ 19439] + * math/Makefile (tests): Add test-math-isinff. + (CFLAGS-test-math-isinff.cc): Use -std=gnu++11. + * math/bits/mathcalls.h [__USE_MISC]: Use + '|| __MATH_DECLARING_DOUBLE == 0' to relax definition of + functions not in C++11 and which don't conflict e.g. isinff, + isinfl etc. + * math/test-math-isinff.cc: New file. + 2016-02-12 Florian Weimer <fweimer@redhat.com> * misc/bug18240.c (do_test): Set RLIMIT_AS. @@ -38,7 +51,7 @@ * misc/Makefile (tests): Add it. 2016-01-28 Steve Ellcey <sellcey@imgtec.com> - Joseph Myers <joseph@codesourcery.com> + Joseph Myers <joseph@codesourcery.com> * sysdeps/mips/memcpy.S (MEMCPY_NAME) [USE_DOUBLE]: Avoid word load in branch delay slot when less than a word of input left. diff --git a/math/Makefile b/math/Makefile index 222ee6b..7d573a0 100644 --- a/math/Makefile +++ b/math/Makefile @@ -114,6 +114,7 @@ tests = test-matherr test-fenv atest-exp atest-sincos atest-exp2 basic-test \ test-nearbyint-except-2 test-signgam-uchar test-signgam-uchar-init \ test-signgam-uint test-signgam-uint-init test-signgam-ullong \ test-signgam-ullong-init test-nan-overflow test-nan-payload \ + test-math-isinff \ $(tests-static) tests-static = test-fpucw-static test-fpucw-ieee-static \ test-signgam-uchar-static test-signgam-uchar-init-static \ @@ -220,6 +221,8 @@ CFLAGS-test-signgam-ullong-init.c = -std=c99 CFLAGS-test-signgam-ullong-static.c = -std=c99 CFLAGS-test-signgam-ullong-init-static.c = -std=c99 +CFLAGS-test-math-isinff.cc = -std=gnu++11 + # The -lieee module sets the _LIB_VERSION_ switch to IEEE mode # for error handling in the -lm functions. install-lib += libieee.a diff --git a/math/bits/mathcalls.h b/math/bits/mathcalls.h index a48345d..9a7b3f0 100644 --- a/math/bits/mathcalls.h +++ b/math/bits/mathcalls.h @@ -196,7 +196,9 @@ __MATHDECL_1 (int,__finite,, (_Mdouble_ __value)) __attribute__ ((__const__)); _Mdouble_END_NAMESPACE #ifdef __USE_MISC -# if !defined __cplusplus || __cplusplus < 201103L /* Conflicts with C++11. */ +# if (!defined __cplusplus \ + || __cplusplus < 201103L /* isinf conflicts with C++11. */ \ + || __MATH_DECLARING_DOUBLE == 0) /* isinff or isinfl don't. */ /* Return 0 if VALUE is finite or NaN, +1 if it is +Infinity, -1 if it is -Infinity. */ __MATHDECL_1 (int,isinf,, (_Mdouble_ __value)) __attribute__ ((__const__)); @@ -232,7 +234,9 @@ __END_NAMESPACE_C99 __MATHDECL_1 (int,__isnan,, (_Mdouble_ __value)) __attribute__ ((__const__)); #if defined __USE_MISC || (defined __USE_XOPEN && !defined __USE_XOPEN2K) -# if !defined __cplusplus || __cplusplus < 201103L /* Conflicts with C++11. */ +# if (!defined __cplusplus \ + || __cplusplus < 201103L /* isnan conflicts with C++11. */ \ + || __MATH_DECLARING_DOUBLE == 0) /* isnanf or isnanl don't. */ /* Return nonzero if VALUE is not a number. */ __MATHDECL_1 (int,isnan,, (_Mdouble_ __value)) __attribute__ ((__const__)); # endif diff --git a/math/test-math-isinff.cc b/math/test-math-isinff.cc new file mode 100644 index 0000000..195d753 --- /dev/null +++ b/math/test-math-isinff.cc @@ -0,0 +1,48 @@ +/* Test for bug 19439. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + Contributed by Marek Polacek <polacek@redhat.com>, 2012. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + <http://www.gnu.org/licenses/>. */ + +#define _GNU_SOURCE 1 +#include <math.h> +#include <stdio.h> +#include <stdlib.h> + +static int +do_test (void) +{ + /* Verify that isinff, isinfl, isnanf, and isnanlf are defined + in the header under C++11 and can be called. Without the + header fix this test will not compile. */ + if (isinff (1.0f) + || !isinff (INFINITY) + || isinfl (1.0L) + || !isinfl (INFINITY) + || isnanf (2.0f) + || !isnanf (NAN) + || isnanl (2.0L) + || !isnanl (NAN)) + { + printf ("FAIL: Failed to call is* functions.\n"); + exit (1); + } + printf ("PASS: Able to call isinff, isinfl, isnanf, and isnanl.\n"); + exit (0); +} + +#define TEST_FUNCTION do_test () +#include "../test-skeleton.c" |