Remove unneeded stubs for k_rem_pio2l.
Commit Message
As noted by Joseph in
https://sourceware.org/ml/libc-alpha/2016-08/msg00927.html
It is simpler to just tweak math/Makefile, and remove the
unused declaration from math_private.h.
---8<---
This is only used for the float and double variants.
Instead, just add it to the type specific list of files,
and remove all stubs, and remove the declaration from
math_private.h.
I verified x86_64, i486, ia64, m68k, and ppc64 build.
* math/Makefile (libm-calls): Remove k_rem_pio2F.
(type-double-routines): Add k_rem_pio2.
(type-float-routines): Add k_rem_pio2f.
* sysdeps/generic/math_private.h:
(__kernel_rem_pio2l): Removed.
* math/k_rem_pio2l.c: Removed.
* sysdeps/i386/fpu/k_rem_pio2l.c: Removed.
* sysdeps/ia64/fpu/k_rem_pio2l.c: Removed.
* sysdeps/m68k/m680x0/fpu/k_rem_pio2l.c: Removed.
* sysdeps/x86_64/fpu/k_rem_pio2l.c: Removed.
---
math/Makefile | 6 +++---
math/k_rem_pio2l.c | 15 ---------------
sysdeps/generic/math_private.h | 2 --
sysdeps/i386/fpu/k_rem_pio2l.c | 3 ---
sysdeps/ia64/fpu/k_rem_pio2l.c | 1 -
sysdeps/m68k/m680x0/fpu/k_rem_pio2l.c | 3 ---
sysdeps/x86_64/fpu/k_rem_pio2l.c | 1 -
7 files changed, 3 insertions(+), 28 deletions(-)
delete mode 100644 math/k_rem_pio2l.c
delete mode 100644 sysdeps/i386/fpu/k_rem_pio2l.c
delete mode 100644 sysdeps/ia64/fpu/k_rem_pio2l.c
delete mode 100644 sysdeps/m68k/m680x0/fpu/k_rem_pio2l.c
delete mode 100644 sysdeps/x86_64/fpu/k_rem_pio2l.c
Comments
On Wed, 31 Aug 2016, Paul E. Murphy wrote:
> * math/Makefile (libm-calls): Remove k_rem_pio2F.
> (type-double-routines): Add k_rem_pio2.
> (type-float-routines): Add k_rem_pio2f.
>
> * sysdeps/generic/math_private.h:
> (__kernel_rem_pio2l): Removed.
>
> * math/k_rem_pio2l.c: Removed.
> * sysdeps/i386/fpu/k_rem_pio2l.c: Removed.
> * sysdeps/ia64/fpu/k_rem_pio2l.c: Removed.
> * sysdeps/m68k/m680x0/fpu/k_rem_pio2l.c: Removed.
> * sysdeps/x86_64/fpu/k_rem_pio2l.c: Removed.
OK.
On 08/31/2016 05:40 PM, Joseph Myers wrote:
> On Wed, 31 Aug 2016, Paul E. Murphy wrote:
>
>> * math/Makefile (libm-calls): Remove k_rem_pio2F.
>> (type-double-routines): Add k_rem_pio2.
>> (type-float-routines): Add k_rem_pio2f.
>>
>> * sysdeps/generic/math_private.h:
>> (__kernel_rem_pio2l): Removed.
>>
>> * math/k_rem_pio2l.c: Removed.
>> * sysdeps/i386/fpu/k_rem_pio2l.c: Removed.
>> * sysdeps/ia64/fpu/k_rem_pio2l.c: Removed.
>> * sysdeps/m68k/m680x0/fpu/k_rem_pio2l.c: Removed.
>> * sysdeps/x86_64/fpu/k_rem_pio2l.c: Removed.
>
> OK.
>
Committed as 2bad840.
@@ -56,7 +56,7 @@ libm-calls = \
e_hypotF e_j0F e_j1F e_jnF e_lgammaF_r e_logF e_log10F e_powF \
e_rem_pio2F e_remainderF e_scalbF e_sinhF e_sqrtF e_gammaF_r \
e_ilogbF \
- k_cosF k_rem_pio2F k_sinF k_tanF s_asinhF s_atanF s_cbrtF \
+ k_cosF k_sinF k_tanF s_asinhF s_atanF s_cbrtF \
s_ceilF s_cosF s_erfF s_expm1F s_fabsF \
s_floorF s_log1pF w_log1pF s_logbF \
s_nextafterF s_nexttowardF s_rintF s_scalblnF w_scalblnF \
@@ -99,11 +99,11 @@ type-ldouble-yes := ldouble
type-double-suffix :=
type-double-routines := branred doasin dosincos halfulp mpa mpatan2 \
mpatan mpexp mplog mpsqrt mptan sincos32 slowexp \
- slowpow sincostab
+ slowpow sincostab k_rem_pio2
# float support
type-float-suffix := f
-type-float-routines :=
+type-float-routines := k_rem_pio2f
# Apply suffix to each type in arg 1
deleted file mode 100644
@@ -1,15 +0,0 @@
-#include <math.h>
-#include <math_private.h>
-#include <stdio.h>
-#include <errno.h>
-
-int
-__kernel_rem_pio2l (long double *x, long double *y, int e0, int nx, int prec,
- const int *ipio2)
-{
- fputs ("__kernel_rem_pio2l not implemented\n", stderr);
- __set_errno (ENOSYS);
- return 0.0;
-}
-
-stub_warning (__kernel_rem_pio2l)
@@ -317,8 +317,6 @@ extern long double __kernel_cosl (long double,long double);
extern long double __kernel_tanl (long double,long double,int);
extern void __kernel_sincosl (long double,long double,
long double *,long double *, int);
-extern int __kernel_rem_pio2l (long double*,long double*,int,int,
- int,const int*);
#ifndef NO_LONG_DOUBLE
/* prototypes required to compile the ldbl-96 support without warnings */
deleted file mode 100644
@@ -1,3 +0,0 @@
-/* Empty. This file is only meant to avoid compiling the file with the
- same name in the libm-ieee754 directory. The code is not used since
- there is an assembler version for all users of this file. */
deleted file mode 100644
@@ -1 +0,0 @@
-/* Not needed. */
deleted file mode 100644
@@ -1,3 +0,0 @@
-/* Empty. This file is only meant to avoid compiling the file with the
- same name in the libm-ieee754 directory. The code is not used since
- there is an assembler version for all users of this file. */
deleted file mode 100644
@@ -1 +0,0 @@
-/* Not needed. */