d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix unused functions in invtrig_test.c

Browse Source 
I only tested the WARNS=6 change on AArch64 and AMD64, but this file has
unused functions for architectures with LDBL_PREC == 53.

While touching this file change the LDBL_PREC == 53 checks to i386 checks.
The long double tests should only be disabled for i386 (due to the rather
odd rounding mode that it uses) not all architectures where long double
is the same as double.

PR:		205449
Fixes:		87d65c747a ("lib/msun: Allow building tests with WARNS=6")
Reported by:	Jenkins
This commit is contained in:
Alex Richardson 2021-03-22 13:05:28 +00:00
parent c8c62548bf
commit 7f5693d053
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/msun/tests/invtrig_test.c
View File

@ -68,7 +68,7 @@ __FBSDID("$FreeBSD$");
(tol) * ldexpf(1.0, 1 - FLT_MANT_DIG), (excepts)); \
} while (0)
#if LDBL_PREC == 53
#ifdef __i386__
#define testall_tol _testall_tol
#else
#define testall_tol(prefix, x, result, tol, excepts) do { \
@ -98,7 +98,7 @@ __FBSDID("$FreeBSD$");
(tol) * ldexpf(1.0, 1 - FLT_MANT_DIG), (excepts)); \
} while (0)
#if LDBL_PREC == 53
#ifdef __i386__
#define testall2_tol _testall2_tol
#else
#define testall2_tol(prefix, y, x, result, tol, excepts) do { \
@ -380,12 +380,14 @@ sinasin(double x)
return (sinl(asin(x)));
}
#ifndef __i386__
static long double
sinasinl(long double x)
{
return (sinl(asinl(x)));
}
#endif
static long double
cosacosf(float x)
@ -401,12 +403,14 @@ cosacos(double x)
return (cosl(acos(x)));
}
#ifndef __i386__
static long double
cosacosl(long double x)
{
return (cosl(acosl(x)));
}
#endif
static long double
tanatanf(float x)
@ -422,12 +426,14 @@ tanatan(double x)
return (tanl(atan(x)));
}
#ifndef __i386__
static long double
tanatanl(long double x)
{
return (tanl(atanl(x)));
}
#endif
ATF_TC_WITHOUT_HEAD(inverse);
ATF_TC_BODY(inverse, tc)