diff options
author | Joseph Myers <joseph@codesourcery.com> | 2017-09-11 23:21:25 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2017-09-11 23:21:25 +0000 |
commit | 4b7d1efb51a529a1391e7326ef90cb51139d7eb6 (patch) | |
tree | 2cbe4418422e009ccb7a94e6fa2fcd28ed7943bd | |
parent | 45ff34638f034877b6a490c217d6a0632ce263f4 (diff) | |
download | glibc-4b7d1efb51a529a1391e7326ef90cb51139d7eb6.tar.gz glibc-4b7d1efb51a529a1391e7326ef90cb51139d7eb6.tar.xz glibc-4b7d1efb51a529a1391e7326ef90cb51139d7eb6.zip |
Define and use a libm_alias_float macro.
Fully supporting TS 18661-3 _FloatN / _FloatNx types in the cases where they have the same format as other supported types (in line with the principles described at <https://sourceware.org/ml/libc-alpha/2017-01/msg00333.html>) means adding a lot of function aliases to libm (and a few to libc). float functions will have *f32 aliases, double functions will have *f32x and *f64 aliases, long double functions may have *f64x, *f128 or both aliases depending on the configuration, float128 functions have have *f64x aliases depending on the configuration. At present, most individual libm functions have their own weak_alias calls to define the public names for those functions. For TS 18661-3 support, it is desirable that functions not all need to duplicate the logic for which alias names to define. Thus, common macros for defining the public aliases to a libm function make sense. In the double and long double cases, such macros will also help simplify existing code (with LONG_DOUBLE_COMPAT etc. conditionals), by eliminating existing conditionals and ldbl-opt / ldbl-64-128 wrappers (using the generated ldbl-compat-choose.h to allow a single macro definition to expand appropriately for each symbol depending on LONG_DOUBLE_COMPAT for that symbol). This patch starts the process of adding such macros with a straightforward case: a libm_alias_float macro, initially only used in the case of type-generic templates, to define aliases for float functions (currently just the *f public names, in future also *f32). Future patches are intended to add such macros for other types and to extend the cases in which they are used, with a view to as many places as possible using them before support for _FloatN / _FloatNx aliases is enabled. (I think it's inevitable that some places doing architecture-specific things with aliases and symbol versioning may end up needing to replicate logic for the new aliases, but hopefully the number of such places can be kept to a minimum.) The libm_alias_float macro takes unsuffixed names for both the internal and public function names. The need for unsuffixed public names is obvious, since such macros will end up defining multiple public names with different suffixes. Unsuffixed internal names are because I expect the ldbl-128 functions to end up in a form that always defines *f128 names and sometimes also defines *l names - with the main internal names being e.g. __ieee754_<func>f128 (so many macros in float128_private.h can go away). But __ieee754_<func>l aliases will still be needed for e.g. use from math/ complex functions, meaning the alias macro needs to see just __ieee754_<func> as internal name so it can create an alias based on that name. Since libm_alias_float128 will thus need the unsuffixed internal name, it seems to make sense for all such macros to receive the unsuffixed name. Tested for x86_64. Also tested with build-many-glibcs.py that installed stripped shared libraries are unchanged by the patch. * sysdeps/generic/libm-alias-float.h: New file. * sysdeps/generic/math-type-macros-float.h: Include <libm-alias-float.h>. [!declare_mgen_alias] (declare_mgen_alias): Define macro.
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | sysdeps/generic/libm-alias-float.h | 35 | ||||
-rw-r--r-- | sysdeps/generic/math-type-macros-float.h | 6 |
3 files changed, 48 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index 952501cbe3..a9b6460090 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2017-09-11 Joseph Myers <joseph@codesourcery.com> + + * sysdeps/generic/libm-alias-float.h: New file. + * sysdeps/generic/math-type-macros-float.h: Include + <libm-alias-float.h>. + [!declare_mgen_alias] (declare_mgen_alias): Define macro. + 2017-09-11 H.J. Lu <hongjiu.lu@intel.com> [BZ #22093] diff --git a/sysdeps/generic/libm-alias-float.h b/sysdeps/generic/libm-alias-float.h new file mode 100644 index 0000000000..23f0166697 --- /dev/null +++ b/sysdeps/generic/libm-alias-float.h @@ -0,0 +1,35 @@ +/* Define aliases for libm float functions. + Copyright (C) 2017 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + 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/>. */ + +#ifndef _LIBM_ALIAS_FLOAT_H +#define _LIBM_ALIAS_FLOAT_H + +/* Define aliases for a float libm function that has internal name + FROM ## f ## R and public names TO ## suffix ## R for each suffix + of a supported floating-point type with the same format as float. + This should only be used for functions where such public names + exist for _FloatN types, not for implementation-namespace exported + names (where there is one name per format, not per type) or for + obsolescent functions not provided for _FloatN types. */ +#define libm_alias_float_r(from, to, r) \ + weak_alias (from ## f ## r, to ## f ## r) + +/* Likewise, but without the R suffix. */ +#define libm_alias_float(from, to) libm_alias_float_r (from, to, ) + +#endif diff --git a/sysdeps/generic/math-type-macros-float.h b/sysdeps/generic/math-type-macros-float.h index cc8c4b0bcd..b23789f678 100644 --- a/sysdeps/generic/math-type-macros-float.h +++ b/sysdeps/generic/math-type-macros-float.h @@ -30,6 +30,12 @@ the double macro constants. */ #define M_MLIT(c) c +#include <libm-alias-float.h> + +#ifndef declare_mgen_alias +# define declare_mgen_alias(from, to) libm_alias_float (from, to) +#endif + /* Supply the generic macros. */ #include <math-type-macros.h> |