mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-29 08:11:08 +00:00
418d99e622
<fenv_private.h> has inline versions of various <fenv.h> functions, and their __fe* variants, for systems (generally soft-float) without support for floating-point exceptions, rounding modes or both. Having these inlines in a separate header introduces a risk of a source file including <fenv.h> and compiling OK on x86_64, but failing to compile (because the feraiseexcept inline is actually a macro that discards its argument, to avoid the need for #ifdef FE_INVALID conditionals), or not being properly optimized, on systems without the exceptions and rounding modes support (when these inlines were in math_private.h, we had a few cases where this broke the build because there was no obvious reason for a file to need math_private.h and it didn't need that header on x86_64). By moving those inlines to include/fenv.h, this risk can be avoided, and fenv_private.h becomes more clearly defined as specifically the header for the internal libc_fe* and SET_RESTORE_ROUND* interfaces. This patch makes that move, removing fenv_private.h includes that are no longer needed (or replacing them by fenv.h includes in a few cases that didn't already have such an include). Tested for x86_64 and x86, and tested with build-many-glibcs.py that installed stripped shared libraries are unchanged by the patch. * sysdeps/generic/fenv_private.h [FE_ALL_EXCEPT == 0]: Move this code .... [!FE_HAVE_ROUNDING_MODES]: And this code .... * include/fenv.h [!_ISOMAC]: ... to here. * math/fraiseexcpt.c (__feraiseexcept): Undefine as macro. (feraiseexcept): Likewise. * math/fromfp.h: Do not include <fenv_private.h>. * math/s_cexp_template.c: Likewise. * math/s_csin_template.c: Likewise. * math/s_csinh_template.c: Likewise. * math/s_ctan_template.c: Likewise. * math/s_ctanh_template.c: Likewise. * math/s_iseqsig_template.c: Likewise. * math/w_acos_compat.c: Likewise. * math/w_acosf_compat.c: Likewise. * math/w_acosl_compat.c: Likewise. * math/w_asin_compat.c: Likewise. * math/w_asinf_compat.c: Likewise. * math/w_asinl_compat.c: Likewise. * math/w_j0_compat.c: Likewise. * math/w_j0f_compat.c: Likewise. * math/w_j0l_compat.c: Likewise. * math/w_j1_compat.c: Likewise. * math/w_j1f_compat.c: Likewise. * math/w_j1l_compat.c: Likewise. * math/w_jn_compat.c: Likewise. * math/w_jnf_compat.c: Likewise. * math/w_log10_compat.c: Likewise. * math/w_log10f_compat.c: Likewise. * math/w_log10l_compat.c: Likewise. * math/w_log2_compat.c: Likewise. * math/w_log2f_compat.c: Likewise. * math/w_log2l_compat.c: Likewise. * math/w_log_compat.c: Likewise. * math/w_logf_compat.c: Likewise. * math/w_logl_compat.c: Likewise. * sysdeps/ieee754/dbl-64/s_llrint.c: Likewise. * sysdeps/ieee754/dbl-64/s_llround.c: Likewise. * sysdeps/ieee754/dbl-64/s_lrint.c: Likewise. * sysdeps/ieee754/dbl-64/s_lround.c: Likewise. * sysdeps/ieee754/dbl-64/wordsize-64/s_lround.c: Likewise. * sysdeps/ieee754/flt-32/s_llrintf.c: Likewise. * sysdeps/ieee754/flt-32/s_llroundf.c: Likewise. * sysdeps/ieee754/flt-32/s_lrintf.c: Likewise. * sysdeps/ieee754/flt-32/s_lroundf.c: Likewise. * sysdeps/ieee754/k_standardl.c: Likewise. * sysdeps/ieee754/ldbl-128/e_expl.c: Likewise. * sysdeps/ieee754/ldbl-128/s_fmal.c: Likewise. * sysdeps/ieee754/ldbl-128/s_llrintl.c: Likewise. * sysdeps/ieee754/ldbl-128/s_llroundl.c: Likewise. * sysdeps/ieee754/ldbl-128/s_lrintl.c: Likewise. * sysdeps/ieee754/ldbl-128/s_lroundl.c: Likewise. * sysdeps/ieee754/ldbl-128/s_nearbyintl.c: Likewise. * sysdeps/ieee754/ldbl-128ibm/s_llrintl.c: Likewise. * sysdeps/ieee754/ldbl-128ibm/s_llroundl.c: Likewise. * sysdeps/ieee754/ldbl-128ibm/s_lrintl.c: Likewise. * sysdeps/ieee754/ldbl-128ibm/s_lroundl.c: Likewise. * sysdeps/ieee754/ldbl-96/s_fma.c: Likewise. * sysdeps/ieee754/ldbl-96/s_fmal.c: Likewise. * sysdeps/ieee754/ldbl-96/s_llrintl.c: Likewise. * sysdeps/ieee754/ldbl-96/s_llroundl.c: Likewise. * sysdeps/ieee754/ldbl-96/s_lrintl.c: Likewise. * sysdeps/ieee754/ldbl-96/s_lroundl.c: Likewise. * math/w_ilogb_template.c: Include <fenv.h> instead of <fenv_private.h>. * math/w_llogb_template.c: Likewise. * sysdeps/powerpc/fpu/e_sqrt.c: Likewise. * sysdeps/powerpc/fpu/e_sqrtf.c: Likewise.
176 lines
5.7 KiB
C
176 lines
5.7 KiB
C
/* Round to integer type. Common helper functions.
|
|
Copyright (C) 2016-2018 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/>. */
|
|
|
|
#include <errno.h>
|
|
#include <fenv.h>
|
|
#include <float.h>
|
|
#include <math.h>
|
|
#include <math-barriers.h>
|
|
#include <math_private.h>
|
|
#include <stdbool.h>
|
|
#include <stdint.h>
|
|
|
|
/* The including file should have defined UNSIGNED to 0 (signed return
|
|
type) or 1 (unsigned return type), INEXACT to 0 (no inexact
|
|
exceptions) or 1 (raise inexact exceptions) and RET_TYPE to the
|
|
return type (intmax_t or uintmax_t). */
|
|
|
|
/* Return the maximum unbiased exponent for an argument (negative if
|
|
NEGATIVE is set) that might be in range for a call to a fromfp
|
|
function with width WIDTH (greater than 0, and not exceeding that
|
|
of intmax_t). The truncated argument may still be out of range in
|
|
the case of negative arguments, and if not out of range it may
|
|
become out of range as a result of rounding. */
|
|
|
|
static int
|
|
fromfp_max_exponent (bool negative, int width)
|
|
{
|
|
if (UNSIGNED)
|
|
return negative ? -1 : width - 1;
|
|
else
|
|
return negative ? width - 1 : width - 2;
|
|
}
|
|
|
|
/* Return the result of rounding an integer value X (passed as the
|
|
absolute value; NEGATIVE is true if the value is negative), where
|
|
HALF_BIT is true if the bit with value 0.5 is set and MORE_BITS is
|
|
true if any lower bits are set, in the rounding direction
|
|
ROUND. */
|
|
|
|
static uintmax_t
|
|
fromfp_round (bool negative, uintmax_t x, bool half_bit, bool more_bits,
|
|
int round)
|
|
{
|
|
switch (round)
|
|
{
|
|
case FP_INT_UPWARD:
|
|
return x + (!negative && (half_bit || more_bits));
|
|
|
|
case FP_INT_DOWNWARD:
|
|
return x + (negative && (half_bit || more_bits));
|
|
|
|
case FP_INT_TOWARDZERO:
|
|
default:
|
|
/* Unknown rounding directions are defined to mean unspecified
|
|
rounding; treat this as truncation. */
|
|
return x;
|
|
|
|
case FP_INT_TONEARESTFROMZERO:
|
|
return x + half_bit;
|
|
|
|
case FP_INT_TONEAREST:
|
|
return x + (half_bit && ((x & 1) || more_bits));
|
|
}
|
|
}
|
|
|
|
/* Integer rounding, of a value whose exponent EXPONENT did not exceed
|
|
the maximum exponent MAX_EXPONENT and so did not necessarily
|
|
overflow, has produced X (possibly wrapping to 0); the sign is
|
|
negative if NEGATIVE is true. Return whether this overflowed the
|
|
allowed width. */
|
|
|
|
static bool
|
|
fromfp_overflowed (bool negative, uintmax_t x, int exponent, int max_exponent)
|
|
{
|
|
if (UNSIGNED)
|
|
{
|
|
if (negative)
|
|
return x != 0;
|
|
else if (max_exponent == INTMAX_WIDTH - 1)
|
|
return exponent == INTMAX_WIDTH - 1 && x == 0;
|
|
else
|
|
return x == (1ULL << (max_exponent + 1));
|
|
}
|
|
else
|
|
{
|
|
if (negative)
|
|
return exponent == max_exponent && x != (1ULL << max_exponent);
|
|
else
|
|
return x == (1ULL << (max_exponent + 1));
|
|
}
|
|
}
|
|
|
|
/* Handle a domain error for a call to a fromfp function with an
|
|
argument which is negative if NEGATIVE is set, and specified width
|
|
(not exceeding that of intmax_t) WIDTH. The return value is
|
|
unspecified (with it being unclear if the result needs to fit
|
|
within WIDTH bits in this case); we choose to saturate to the given
|
|
number of bits (treating NaNs like any other value). */
|
|
|
|
static RET_TYPE
|
|
fromfp_domain_error (bool negative, unsigned int width)
|
|
{
|
|
feraiseexcept (FE_INVALID);
|
|
__set_errno (EDOM);
|
|
/* The return value is unspecified; we choose to saturate to the
|
|
given number of bits (treating NaNs like any other value). */
|
|
if (UNSIGNED)
|
|
{
|
|
if (negative)
|
|
return 0;
|
|
else if (width == INTMAX_WIDTH)
|
|
return -1;
|
|
else
|
|
return (1ULL << width) - 1;
|
|
}
|
|
else
|
|
{
|
|
if (width == 0)
|
|
return 0;
|
|
else if (negative)
|
|
return -(1ULL << (width - 1));
|
|
else
|
|
return (1ULL << (width - 1)) - 1;
|
|
}
|
|
}
|
|
|
|
/* Given X, the absolute value of a floating-point number (negative if
|
|
NEGATIVE is set) truncated towards zero, where HALF_BIT is true if
|
|
the bit with value 0.5 is set and MORE_BITS is true if any lower
|
|
bits are set, round it in the rounding direction ROUND, handle
|
|
errors and exceptions and return the appropriate return value for a
|
|
fromfp function. X originally had floating-point exponent
|
|
EXPONENT, which does not exceed MAX_EXPONENT, the return value from
|
|
fromfp_max_exponent with width WIDTH. */
|
|
|
|
static RET_TYPE
|
|
fromfp_round_and_return (bool negative, uintmax_t x, bool half_bit,
|
|
bool more_bits, int round, int exponent,
|
|
int max_exponent, unsigned int width)
|
|
{
|
|
uintmax_t uret = fromfp_round (negative, x, half_bit, more_bits, round);
|
|
if (fromfp_overflowed (negative, uret, exponent, max_exponent))
|
|
return fromfp_domain_error (negative, width);
|
|
|
|
if (INEXACT && (half_bit || more_bits))
|
|
{
|
|
/* There is no need for this to use the specific floating-point
|
|
type for which this header is included, and there is no need
|
|
for this header to know that type at all, so just use float
|
|
here. */
|
|
float force_inexact = 1.0f + FLT_MIN;
|
|
math_force_eval (force_inexact);
|
|
}
|
|
if (UNSIGNED)
|
|
/* A negative argument not rounding to zero will already have
|
|
produced a domain error. */
|
|
return uret;
|
|
else
|
|
return negative ? -uret : uret;
|
|
}
|