[02/13] aarch64: Use math-use-builtins for ceil{f}

Message ID 20200609213301.3591135-2-adhemerval.zanella@linaro.org
State New
Headers show
Series
  • [01/13] math: Decompose math-use-builtins.h
Related show

Commit Message

The define is already set on the math-use-builtins-ceil.h, the patch
just removes the implementations (it was missed on c9feb1be93).

Checked on aarch64-linux-gnu.
---
 sysdeps/aarch64/fpu/s_ceil.c  | 29 -----------------------------
 sysdeps/aarch64/fpu/s_ceilf.c | 29 -----------------------------
 2 files changed, 58 deletions(-)
 delete mode 100644 sysdeps/aarch64/fpu/s_ceil.c
 delete mode 100644 sysdeps/aarch64/fpu/s_ceilf.c

-- 
2.25.1

Comments

Szabolcs Nagy June 10, 2020, 10:15 a.m. | #1
The 06/09/2020 18:32, Adhemerval Zanella via Libc-alpha wrote:
> The define is already set on the math-use-builtins-ceil.h, the patch

> just removes the implementations (it was missed on c9feb1be93).

> 

> Checked on aarch64-linux-gnu.

> ---

>  sysdeps/aarch64/fpu/s_ceil.c  | 29 -----------------------------

>  sysdeps/aarch64/fpu/s_ceilf.c | 29 -----------------------------

>  2 files changed, 58 deletions(-)

>  delete mode 100644 sysdeps/aarch64/fpu/s_ceil.c

>  delete mode 100644 sysdeps/aarch64/fpu/s_ceilf.c


this is ok to commit.

Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>

Patch

diff --git a/sysdeps/aarch64/fpu/s_ceil.c b/sysdeps/aarch64/fpu/s_ceil.c
deleted file mode 100644
index f3cefdc824..0000000000
--- a/sysdeps/aarch64/fpu/s_ceil.c
+++ /dev/null
@@ -1,29 +0,0 @@ 
-/* Copyright (C) 2011-2020 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
-   <https://www.gnu.org/licenses/>.  */
-
-#define NO_MATH_REDIRECT
-#include <math.h>
-#include <libm-alias-double.h>
-
-double
-__ceil (double x)
-{
-  return __builtin_ceil (x);
-}
-
-libm_alias_double (__ceil, ceil)
diff --git a/sysdeps/aarch64/fpu/s_ceilf.c b/sysdeps/aarch64/fpu/s_ceilf.c
deleted file mode 100644
index 1cde85fbaa..0000000000
--- a/sysdeps/aarch64/fpu/s_ceilf.c
+++ /dev/null
@@ -1,29 +0,0 @@ 
-/* Copyright (C) 2011-2020 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
-   <https://www.gnu.org/licenses/>.  */
-
-#define NO_MATH_REDIRECT
-#include <math.h>
-#include <libm-alias-float.h>
-
-float
-__ceilf (float x)
-{
-  return __builtin_ceilf (x);
-}
-
-libm_alias_float (__ceil, ceil)