From patchwork Tue Aug 30 23:39:37 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Paul E. Murphy" X-Patchwork-Id: 15106 X-Patchwork-Delegate: joseph@codesourcery.com Received: (qmail 35118 invoked by alias); 30 Aug 2016 23:39:55 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 35016 invoked by uid 89); 30 Aug 2016 23:39:54 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.1 required=5.0 tests=AWL, BAYES_00, KAM_LAZY_DOMAIN_SECURITY, RCVD_IN_DNSWL_LOW, RCVD_IN_SEMBACKSCATTER autolearn=no version=3.3.2 spammy=drepper, UD:math_ldbl_opt.h, math_ldbl_opth, math_ldbl_opt.h X-HELO: mx0a-001b2d01.pphosted.com X-IBM-Helo: d01dlp03.pok.ibm.com X-IBM-MailFrom: murphyp@linux.vnet.ibm.com From: "Paul E. Murphy" To: libc-alpha@sourceware.org Subject: [PATCH 4/5] Make common fmin implementation generic. Date: Tue, 30 Aug 2016 18:39:37 -0500 In-Reply-To: References: In-Reply-To: References: X-TM-AS-GCONF: 00 X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16083023-0044-0000-0000-0000010EF089 X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00005680; HX=3.00000240; KW=3.00000007; PH=3.00000004; SC=3.00000184; SDB=6.00751825; UDB=6.00355389; IPR=6.00524631; BA=6.00004681; NDR=6.00000001; ZLA=6.00000005; ZF=6.00000009; ZB=6.00000000; ZP=6.00000000; ZH=6.00000000; ZU=6.00000002; MB=3.00012523; XFM=3.00000011; UTC=2016-08-30 23:39:43 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 16083023-0045-0000-0000-00000525F6AE Message-Id: <5bab985bb0c6156137f3925101b15bbb89c947ca.1472599898.git.murphyp@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2016-08-30_10:, , signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=4 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1604210000 definitions=main-1608300225 * math/Makefile (gen-libm-calls): Add s_fmin (libm-calls): Remove above. * math/s_fmin.c: Refactor into ... * math/s_fmin_template.c: New file. * math/s_fminf.c: Removed. * math/s_fminl.c: Removed. * sysdeps/ieee754/ldbl-opt/math-type-macros-double.h: (LDOUBLE_fmin_libm_version): New macro. * sysdeps/ieee754/ldbl-opt/s_fmin.c: Removed. * sysdeps/ieee754/ldbl-opt/s_fminl.c: Removed. --- math/Makefile | 4 +-- math/s_fmin.c | 32 ---------------------- math/s_fmin_template.c | 32 ++++++++++++++++++++++ math/s_fminf.c | 28 ------------------- math/s_fminl.c | 28 ------------------- sysdeps/ieee754/ldbl-opt/math-type-macros-double.h | 1 + sysdeps/ieee754/ldbl-opt/s_fmin.c | 5 ---- sysdeps/ieee754/ldbl-opt/s_fminl.c | 5 ---- 8 files changed, 35 insertions(+), 100 deletions(-) delete mode 100644 math/s_fmin.c create mode 100644 math/s_fmin_template.c delete mode 100644 math/s_fminf.c delete mode 100644 math/s_fminl.c delete mode 100644 sysdeps/ieee754/ldbl-opt/s_fmin.c delete mode 100644 sysdeps/ieee754/ldbl-opt/s_fminl.c diff --git a/math/Makefile b/math/Makefile index 09c4c56..c0d7ff0 100644 --- a/math/Makefile +++ b/math/Makefile @@ -49,7 +49,7 @@ gen-libm-calls = cargF conjF cimagF crealF cabsF s_cacosF \ s_cacoshF s_ccosF s_ccoshF s_casinF s_csinF s_casinhF \ k_casinhF s_csinhF k_casinhF s_csinhF s_catanhF s_catanF \ s_ctanF s_ctanhF s_cexpF s_clogF s_cprojF s_csqrtF \ - s_cpowF s_clog10F s_fdimF s_nextdownF s_fmaxF + s_cpowF s_clog10F s_fdimF s_nextdownF s_fmaxF s_fminF libm-calls = \ e_acosF e_acoshF e_asinF e_atan2F e_atanhF e_coshF e_expF e_fmodF \ @@ -65,7 +65,7 @@ libm-calls = \ w_tgammaF w_hypotF w_j0F w_j1F w_jnF w_lgammaF w_lgammaF_r \ w_logF w_log10F w_powF w_remainderF w_scalbF w_sinhF w_sqrtF \ w_ilogbF \ - s_fpclassifyF s_fminF s_nanF s_truncF \ + s_fpclassifyF s_nanF s_truncF \ s_remquoF e_log2F e_exp2F s_roundF s_nearbyintF s_sincosF \ s_fmaF s_lrintF s_llrintF s_lroundF s_llroundF e_exp10F w_log2F \ s_issignalingF $(calls:s_%=m_%) x2y2m1F \ diff --git a/math/s_fmin.c b/math/s_fmin.c deleted file mode 100644 index d22b916..0000000 --- a/math/s_fmin.c +++ /dev/null @@ -1,32 +0,0 @@ -/* Return minimum numeric value of X and Y. - Copyright (C) 1997-2016 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Ulrich Drepper , 1997. - - 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 - . */ - -#include - - -double -__fmin (double x, double y) -{ - return (islessequal (x, y) || isnan (y)) ? x : y; -} -weak_alias (__fmin, fmin) -#ifdef NO_LONG_DOUBLE -strong_alias (__fmin, __fminl) -weak_alias (__fmin, fminl) -#endif diff --git a/math/s_fmin_template.c b/math/s_fmin_template.c new file mode 100644 index 0000000..b70989a --- /dev/null +++ b/math/s_fmin_template.c @@ -0,0 +1,32 @@ +/* Return minimum numeric value of X and Y. + Copyright (C) 1997-2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + Contributed by Ulrich Drepper , 1997. + + 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 + . */ + +#include + + +FLOAT +M_DECL_FUNC (__fmin) (FLOAT x, FLOAT y) +{ + return (islessequal (x, y) || isnan (y)) ? x : y; +} +declare_mgen_alias (__fmin, fmin); + +#if M_LIBM_NEED_COMPAT (fmin) +declare_mgen_libm_compat (__fmin, fmin) +#endif diff --git a/math/s_fminf.c b/math/s_fminf.c deleted file mode 100644 index 968373a..0000000 --- a/math/s_fminf.c +++ /dev/null @@ -1,28 +0,0 @@ -/* Return minimum numeric value of X and Y. - Copyright (C) 1997-2016 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Ulrich Drepper , 1997. - - 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 - . */ - -#include - - -float -__fminf (float x, float y) -{ - return (islessequal (x, y) || isnan (y)) ? x : y; -} -weak_alias (__fminf, fminf) diff --git a/math/s_fminl.c b/math/s_fminl.c deleted file mode 100644 index 84e9e51..0000000 --- a/math/s_fminl.c +++ /dev/null @@ -1,28 +0,0 @@ -/* Return minimum numeric value of X and Y. - Copyright (C) 1997-2016 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Ulrich Drepper , 1997. - - 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 - . */ - -#include - - -long double -__fminl (long double x, long double y) -{ - return (islessequal (x, y) || isnan (y)) ? x : y; -} -weak_alias (__fminl, fminl) diff --git a/sysdeps/ieee754/ldbl-opt/math-type-macros-double.h b/sysdeps/ieee754/ldbl-opt/math-type-macros-double.h index 5644df0..b5136b7 100644 --- a/sysdeps/ieee754/ldbl-opt/math-type-macros-double.h +++ b/sysdeps/ieee754/ldbl-opt/math-type-macros-double.h @@ -46,6 +46,7 @@ #define LDOUBLE___clog10l_libm_version GLIBC_2_1 #define LDOUBLE_fdiml_libm_version GLIBC_2_1 #define LDOUBLE_fmaxl_libm_version GLIBC_2_1 +#define LDOUBLE_fminl_libm_version GLIBC_2_1 /* Define compat symbols for long double on platforms where it was not always a distinct type. */ diff --git a/sysdeps/ieee754/ldbl-opt/s_fmin.c b/sysdeps/ieee754/ldbl-opt/s_fmin.c deleted file mode 100644 index c3fe44d..0000000 --- a/sysdeps/ieee754/ldbl-opt/s_fmin.c +++ /dev/null @@ -1,5 +0,0 @@ -#include -#include -#if LONG_DOUBLE_COMPAT(libm, GLIBC_2_1) -compat_symbol (libm, __fmin, fminl, GLIBC_2_1); -#endif diff --git a/sysdeps/ieee754/ldbl-opt/s_fminl.c b/sysdeps/ieee754/ldbl-opt/s_fminl.c deleted file mode 100644 index 9bfdc7a..0000000 --- a/sysdeps/ieee754/ldbl-opt/s_fminl.c +++ /dev/null @@ -1,5 +0,0 @@ -#include -#undef weak_alias -#define weak_alias(n,a) -#include -long_double_symbol (libm, __fminl, fminl);