diff options
author | Joseph Myers <joseph@codesourcery.com> | 2017-10-30 20:58:51 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2017-10-30 20:58:51 +0000 |
commit | eda162dd8a7d2ca0b1900aa57a78b9586800b4ec (patch) | |
tree | 18789ab225d4de045819be9a61de3bef4f1a3cbc | |
parent | 4ad5106e3b04cc7630f7dbfdb25369807f532843 (diff) | |
download | glibc-eda162dd8a7d2ca0b1900aa57a78b9586800b4ec.tar.gz glibc-eda162dd8a7d2ca0b1900aa57a78b9586800b4ec.tar.xz glibc-eda162dd8a7d2ca0b1900aa57a78b9586800b4ec.zip |
Remove _Mlong_double_ etc. macros.
math.h has a macro _Mlong_double_ for the type to use when declaring long double functions, and similar macros for other types. math/Makefile uses -D_Mlong_double_=double in the case of long double having the same ABI as double. This originates with: Mon Jul 8 13:37:40 1996 Roland McGrath <roland@delasyd.gnu.ai.mit.edu> * math/math.h (_Mfloat_, _Mlong_double_): New macros, defined iff not already defined to float, long double. Use those macros for _Mdouble_ defns when including mathcalls.h. * math/Makefile [$(long-double-fcts) != yes] (CPPFLAGS): Append -D_Mlong_double_=double. However, math.h stopped declaring long double functions in the case of long double having the same ABI as double (and thus probably stopped actually needing the Makefile definition of _Mlong_double_) with: 1998-11-05 Ulrich Drepper <drepper@cygnus.com> * math/math.h: Unconditionally include bits/mathdef.h. Declare long double functions only if __NO_LONG_DOUBLE_MATH is not defined. * sysdeps/generic/bits/mathdef.h: Define only if __USE_ISOC9X. Define __NO_LONG_DOUBLE_MATH. * sysdeps/m68k/fpu/bits/mathdef.h: Define only if __USE_ISOC9X. * sysdeps/i386/fpu/bits/mathdef.h: Likewise. The declarations were since restored for compiling user code, but remain absent when _LIBC is defined, which is sufficient to avoid problems declaring function aliases of incompatible types. Thus the indirection through the _Mlong_double_ macro is not needed (probably since that 1998 patch), and this patch removes _Mlong_double_ and associated macros for other types, leaving only the macro _Mdouble_ which is actually used as the type for which a given inclusion of <bits/mathcalls.h> should declared functions. Tested for x86_64, and tested with build-many-glibcs.py that installed stripped shared libraries are unchanged by this patch. * math/math.h [!_Mfloat_] (_Mfloat_): Do not define. [!_Mlong_double_] (_Mlong_double_): Likewise. [!_Mfloat16_] (_Mfloat16_): Likewise. [!_Mfloat32_] (_Mfloat32_): Likewise. [!_Mfloat64_] (_Mfloat64_): Likewise. [!_Mfloat128_] (_Mfloat128_): Likewise. [!_Mfloat32x_] (_Mfloat32x_): Likewise. [!_Mfloat64x_] (_Mfloat64x_): Likewise. [!_Mfloat128x_] (_Mfloat128x_): Likewise. (_Mdouble_): Define without indirection through those macros. * math/complex.h [!_Mfloat_] (_Mfloat_): Do not define. [!_Mfloat128_] (_Mfloat128_): Likewise. [_Mlong_double_] (_Mlong_double_): Likewise. (_Mdouble_): Define without indirection through those macros. * math/Makefile [$(long-double-fcts) != yes] (math-CPPFLAGS): Do not add -D_Mlong_double_=double. * include/math.h [_ISOMAC] (_Mlong_double_): Do not undefine. * math/test-signgam-finite-c99.c (_Mlong_double_): Likewise.
-rw-r--r-- | ChangeLog | 21 | ||||
-rw-r--r-- | include/math.h | 1 | ||||
-rw-r--r-- | math/Makefile | 6 | ||||
-rw-r--r-- | math/complex.h | 15 | ||||
-rw-r--r-- | math/math.h | 45 | ||||
-rw-r--r-- | math/test-signgam-finite-c99.c | 1 |
6 files changed, 35 insertions, 54 deletions
diff --git a/ChangeLog b/ChangeLog index c99898e127..395aacf7ad 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,24 @@ +2017-10-30 Joseph Myers <joseph@codesourcery.com> + + * math/math.h [!_Mfloat_] (_Mfloat_): Do not define. + [!_Mlong_double_] (_Mlong_double_): Likewise. + [!_Mfloat16_] (_Mfloat16_): Likewise. + [!_Mfloat32_] (_Mfloat32_): Likewise. + [!_Mfloat64_] (_Mfloat64_): Likewise. + [!_Mfloat128_] (_Mfloat128_): Likewise. + [!_Mfloat32x_] (_Mfloat32x_): Likewise. + [!_Mfloat64x_] (_Mfloat64x_): Likewise. + [!_Mfloat128x_] (_Mfloat128x_): Likewise. + (_Mdouble_): Define without indirection through those macros. + * math/complex.h [!_Mfloat_] (_Mfloat_): Do not define. + [!_Mfloat128_] (_Mfloat128_): Likewise. + [_Mlong_double_] (_Mlong_double_): Likewise. + (_Mdouble_): Define without indirection through those macros. + * math/Makefile [$(long-double-fcts) != yes] (math-CPPFLAGS): Do + not add -D_Mlong_double_=double. + * include/math.h [_ISOMAC] (_Mlong_double_): Do not undefine. + * math/test-signgam-finite-c99.c (_Mlong_double_): Likewise. + 2017-10-30 H.J. Lu <hongjiu.lu@intel.com> * sysdeps/x86/libc-start.c: Add /* !SHARED */. diff --git a/include/math.h b/include/math.h index fe3ed1378d..7ee291fc97 100644 --- a/include/math.h +++ b/include/math.h @@ -2,7 +2,6 @@ #ifdef _ISOMAC # undef NO_LONG_DOUBLE -# undef _Mlong_double_ #endif #include <math/math.h> diff --git a/math/Makefile b/math/Makefile index 1feb42561e..b2bd3d3bcc 100644 --- a/math/Makefile +++ b/math/Makefile @@ -561,10 +561,8 @@ math-CPPFLAGS += -D__NO_MATH_INLINES -D__LIBC_INTERNAL_MATH_INLINES ifneq ($(long-double-fcts),yes) # The `double' and `long double' types are the same on this machine. # We won't compile the `long double' code at all. Tell the `double' code -# to define aliases for the `FUNCl' names. To avoid type conflicts in -# defining those aliases, tell <math.h> to declare the `FUNCl' names with -# `double' instead of `long double'. -math-CPPFLAGS += -DNO_LONG_DOUBLE -D_Mlong_double_=double +# to define aliases for the `FUNCl' names. +math-CPPFLAGS += -DNO_LONG_DOUBLE endif # These files quiet sNaNs in a way that is optimized away without diff --git a/math/complex.h b/math/complex.h index 1ae4ec2409..9a6cc3e799 100644 --- a/math/complex.h +++ b/math/complex.h @@ -83,10 +83,7 @@ __BEGIN_DECLS #undef __MATH_PRECNAME /* Now the float versions. */ -#ifndef _Mfloat_ -# define _Mfloat_ float -#endif -#define _Mdouble_ _Mfloat_ +#define _Mdouble_ float #define __MATH_PRECNAME(name) name##f #include <bits/cmathcalls.h> #undef _Mdouble_ @@ -94,15 +91,12 @@ __BEGIN_DECLS #if (__HAVE_DISTINCT_FLOAT128 || (__HAVE_FLOAT128 && !defined _LIBC)) \ && __GLIBC_USE (IEC_60559_TYPES_EXT) -# ifndef _Mfloat128_ -# define _Mfloat128_ _Float128 -# endif /* GCC < 7 requires extra convincing to expose a complex float128 type. */ # ifdef __CFLOAT128 # undef _Mdouble_complex_ # define _Mdouble_complex_ __CFLOAT128 # endif -# define _Mdouble_ _Mfloat128_ +# define _Mdouble_ _Float128 # define __MATH_PRECNAME(name) name##f128 # include <bits/cmathcalls.h> # undef _Mdouble_ @@ -120,10 +114,7 @@ __BEGIN_DECLS extern type __REDIRECT_NTH(__MATH_PRECNAME(function), args, function) # endif -# ifndef _Mlong_double_ -# define _Mlong_double_ long double -# endif -# define _Mdouble_ _Mlong_double_ +# define _Mdouble_ long double # define __MATH_PRECNAME(name) name##l # include <bits/cmathcalls.h> #endif diff --git a/math/math.h b/math/math.h index 5683ac96fe..88fd4bce54 100644 --- a/math/math.h +++ b/math/math.h @@ -299,10 +299,7 @@ enum /* Include the file of declarations again, this time using `float' instead of `double' and appending f to each function name. */ -# ifndef _Mfloat_ -# define _Mfloat_ float -# endif -# define _Mdouble_ _Mfloat_ +# define _Mdouble_ float # define __MATH_PRECNAME(name,r) name##f##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 0 @@ -344,10 +341,7 @@ extern long double __REDIRECT_NTH (nexttowardl, /* Include the file of declarations again, this time using `long double' instead of `double' and appending l to each function name. */ -# ifndef _Mlong_double_ -# define _Mlong_double_ long double -# endif -# define _Mdouble_ _Mlong_double_ +# define _Mdouble_ long double # define __MATH_PRECNAME(name,r) name##l##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 0 @@ -367,10 +361,7 @@ extern long double __REDIRECT_NTH (nexttowardl, types. */ #if __HAVE_DISTINCT_FLOAT16 || (__HAVE_FLOAT16 && !defined _LIBC) -# ifndef _Mfloat16_ -# define _Mfloat16_ _Float16 -# endif -# define _Mdouble_ _Mfloat16_ +# define _Mdouble_ _Float16 # define __MATH_PRECNAME(name,r) name##f16##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -387,10 +378,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT16 || (__HAVE_FLOAT16 && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT32 || (__HAVE_FLOAT32 && !defined _LIBC) -# ifndef _Mfloat32_ -# define _Mfloat32_ _Float32 -# endif -# define _Mdouble_ _Mfloat32_ +# define _Mdouble_ _Float32 # define __MATH_PRECNAME(name,r) name##f32##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -407,10 +395,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT32 || (__HAVE_FLOAT32 && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT64 || (__HAVE_FLOAT64 && !defined _LIBC) -# ifndef _Mfloat64_ -# define _Mfloat64_ _Float64 -# endif -# define _Mdouble_ _Mfloat64_ +# define _Mdouble_ _Float64 # define __MATH_PRECNAME(name,r) name##f64##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -427,10 +412,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT64 || (__HAVE_FLOAT64 && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT128 || (__HAVE_FLOAT128 && !defined _LIBC) -# ifndef _Mfloat128_ -# define _Mfloat128_ _Float128 -# endif -# define _Mdouble_ _Mfloat128_ +# define _Mdouble_ _Float128 # define __MATH_PRECNAME(name,r) name##f128##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -447,10 +429,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT128 || (__HAVE_FLOAT128 && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT32X || (__HAVE_FLOAT32X && !defined _LIBC) -# ifndef _Mfloat32x_ -# define _Mfloat32x_ _Float32x -# endif -# define _Mdouble_ _Mfloat32x_ +# define _Mdouble_ _Float32x # define __MATH_PRECNAME(name,r) name##f32x##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -467,10 +446,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT32X || (__HAVE_FLOAT32X && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT64X || (__HAVE_FLOAT64X && !defined _LIBC) -# ifndef _Mfloat64x_ -# define _Mfloat64x_ _Float64x -# endif -# define _Mdouble_ _Mfloat64x_ +# define _Mdouble_ _Float64x # define __MATH_PRECNAME(name,r) name##f64x##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 @@ -487,10 +463,7 @@ extern long double __REDIRECT_NTH (nexttowardl, #endif /* __HAVE_DISTINCT_FLOAT64X || (__HAVE_FLOAT64X && !_LIBC). */ #if __HAVE_DISTINCT_FLOAT128X || (__HAVE_FLOAT128X && !defined _LIBC) -# ifndef _Mfloat128x_ -# define _Mfloat128x_ _Float128x -# endif -# define _Mdouble_ _Mfloat128x_ +# define _Mdouble_ _Float128x # define __MATH_PRECNAME(name,r) name##f128x##r # define __MATH_DECLARING_DOUBLE 0 # define __MATH_DECLARING_FLOATN 1 diff --git a/math/test-signgam-finite-c99.c b/math/test-signgam-finite-c99.c index 07fdcda9b3..cfe9df0d9b 100644 --- a/math/test-signgam-finite-c99.c +++ b/math/test-signgam-finite-c99.c @@ -19,7 +19,6 @@ #undef _LIBC #undef __LIBC_INTERNAL_MATH_INLINES #undef _GNU_SOURCE -#undef _Mlong_double_ #include <math.h> #include <stdio.h> |