Remove old cimag inlines

Message ID DB6PR0801MB20532C5DB6D56F46A7BE21E5837E0@DB6PR0801MB2053.eurprd08.prod.outlook.com
State New, archived
Headers

Commit Message

Wilco Dijkstra Sept. 29, 2017, 12:58 p.m. UTC
  Remove some old cimag inlines that are only used in GCC 2.96 or
older.

GLIBC still builds - obviously no effect on binaries, OK for commit?

ChangeLog:
2017-09-29  Wilco Dijkstra  <wdijkstr@arm.com>

	* math/bits/cmathcalls.h (cimag): Remove inline.
	(creal): Remove inline.
	(conj): Remove inline.
--
  

Comments

Joseph Myers Sept. 29, 2017, 4:23 p.m. UTC | #1
On Fri, 29 Sep 2017, Wilco Dijkstra wrote:

> Remove some old cimag inlines that are only used in GCC 2.96 or
> older.
> 
> GLIBC still builds - obviously no effect on binaries, OK for commit?

OK.
  

Patch

diff --git a/math/bits/cmathcalls.h b/math/bits/cmathcalls.h
index 7f042dbeaa89dff630d86e65ae2c275d52733e7a..bf9cc08a7f4dba38692ffe51676ecdf747269f9b 100644
--- a/math/bits/cmathcalls.h
+++ b/math/bits/cmathcalls.h
@@ -129,32 +129,3 @@  __MATHDECL (_Mdouble_,cimag, (_Mdouble_complex_ __z));
 /* Real part of Z.  */
 __MATHDECL (_Mdouble_,creal, (_Mdouble_complex_ __z));
 
-
-/* Now some optimized versions.  GCC has handy notations for these
-   functions.  Recent GCC handles these as builtin functions so does
-   not need inlines.  */
-#if defined __GNUC__ && !__GNUC_PREREQ (2, 97) && defined __OPTIMIZE__ \
-    && defined __extern_inline
-
-/* Imaginary part of Z.  */
-__extern_inline _Mdouble_
-__MATH_PRECNAME(cimag) (_Mdouble_complex_ __z) __THROW
-{
-  return __imag__ __z;
-}
-
-/* Real part of Z.  */
-__extern_inline _Mdouble_
-__MATH_PRECNAME(creal) (_Mdouble_complex_ __z) __THROW
-{
-  return __real__ __z;
-}
-
-/* Complex conjugate of Z.  */
-__extern_inline _Mdouble_complex_
-__MATH_PRECNAME(conj) (_Mdouble_complex_ __z) __THROW
-{
-  return __extension__ ~__z;
-}
-
-#endif