Remove x86 mathinline.h hypot inline.

Continuing the removal of bits/mathinline.h inlines that would better
be done by the compiler, this patch removes an x86 inline for hypot
functions (only for fast-math, only for non-SSE 32-bit x86).  I've
filed <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88474> for adding
such an inline as an optimization in GCC.

Tested for x86_64 and x86.

	* sysdeps/x86/fpu/bits/mathinline.h (hypot): Remove inline
	definition.

diff --git a/ChangeLog b/ChangeLog
index 1b0fff4..92683cb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-12  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/x86/fpu/bits/mathinline.h (hypot): Remove inline
+       definition.
+
 2018-12-12  Leonardo Sandoval  <leonardo.sandoval.gonzalez@intel.com>
 
        * benchtests/scripts/compare_bench.py (do_compare): write to

diff --git a/sysdeps/x86/fpu/bits/mathinline.h b/sysdeps/x86/fpu/bits/mathinline.h
index 91ece8d..8eae370 100644 (file)
--- a/sysdeps/x86/fpu/bits/mathinline.h
+++ b/sysdeps/x86/fpu/bits/mathinline.h
@@ -298,10 +298,6 @@ __inline_mathcodeNP (atanh, __x, \
   register long double __y = __fabsl (__x);                                  \
   return -0.5 * log1pl (-(__y + __y) / (1.0 + __y)) * __sgn1l (__x))
 
-/* The argument range of the inline version of hypotl is slightly reduced.  */
-__inline_mathcodeNP2 (hypot, __x, __y,
-                     return __libc_sqrtl (__x * __x + __y * __y))
-
 #   endif
 #  endif