diff mbox

[7/N,x86_64] Vectorized math functions

Message ID CAMXFM3v_yOYuEm-muDbnOZphnwXYWDjQK9oj1Ln8mFy38upLPQ@mail.gmail.com
State Superseded
Headers

Commit Message

Andrew Senkevich Nov. 25, 2014, 5:13 p.m. UTC 
  This patch adds infrastructure used for addition of SIMD declarations
for math functions in math.h.
Added new headers math-vector.h (only generic version for now) and
libm-simd-decl-stubs.h with empty definitions required for proper
unfolding of new macros __MATHCALL_VEC which will be used for
declaration of vector math functions.

ChangeLog

2014-11-25  Andrew Senkevich  <andrew.n.senkevich@gmail.com>

        * bits/math-vector.h: New file.
        * include/libm-simd-decl-stubs.h: New header.
        * math/Makefile (headers): Added new header libm-simd-decl-stubs.h.
        * math/math.h (__MATHCALL_VEC): New macro.



--
WBR,
Andrew

Comments

Joseph Myers Nov. 28, 2014, 11:40 p.m. UTC | #1 
On Tue, 25 Nov 2014, Andrew Senkevich wrote:

> diff --git a/include/libm-simd-decl-stubs.h b/include/libm-simd-decl-stubs.h

As a header not intended to be directly included by users, this should be 
in bits/, and have the standard checks against direct inclusion / #error 
if the relevant header that should include it hasn't been included.

> +#define __SIMD_DECL(function) __CONCAT (__DECL_SIMD_,function)
> +
> +#define __MATHCALL_VEC(function,suffix, args) \
> +  __SIMD_DECL (__MATH_PRECNAME (function,suffix)) \
> +  __MATHCALL (function,suffix, args)

There should be spaces after commas consistently.
diff mbox

Patch

diff --git a/bits/math-vector.h b/bits/math-vector.h
new file mode 100644
index 0000000..c8fe5cb
--- /dev/null
+++ b/bits/math-vector.h
@@ -0,0 +1,22 @@ 
+/* Platform-specific SIMD declarations of math functions.
+   Copyright (C) 2014 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  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/>.  */
+
+#ifndef _MATH_H
+# error "Never include <bits/math-vector.h> directly; \
+ include <math.h> instead."
+#endif
diff --git a/include/libm-simd-decl-stubs.h b/include/libm-simd-decl-stubs.h
new file mode 100644
index 0000000..0048717
--- /dev/null
+++ b/include/libm-simd-decl-stubs.h
@@ -0,0 +1,35 @@ 
+/* Empty definitions required for __MATHCALL_VEC unfolding in mathcalls.h.
+   Copyright (C) 2014 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/>.  */
+
+/* Needed definitions could be generated with:
+   for func in $(grep __MATHCALL_VEC math/bits/mathcalls.h |\
+ sed -r "s|__MATHCALL_VEC.?\(||; s|,.*||"); do
+     echo "#define __DECL_SIMD_${func}";
+     echo "#define __DECL_SIMD_${func}f";
+     echo "#define __DECL_SIMD_${func}l";
+   done
+ */
+
+#ifndef _LIBM_SIMD_DECL_STUBS_H
+#define _LIBM_SIMD_DECL_STUBS_H 1
+
+#define __DECL_SIMD_cos
+#define __DECL_SIMD_cosf
+#define __DECL_SIMD_cosl
+
+#endif
diff --git a/math/Makefile b/math/Makefile
index a2240d0..f5a23e4 100644
--- a/math/Makefile
+++ b/math/Makefile
@@ -26,7 +26,7 @@  headers := math.h bits/mathcalls.h bits/mathinline.h
bits/huge_val.h \
    bits/huge_valf.h bits/huge_vall.h bits/inf.h bits/nan.h \
    fpu_control.h complex.h bits/cmathcalls.h fenv.h \
    bits/fenv.h bits/fenvinline.h bits/mathdef.h tgmath.h \
-   bits/math-finite.h
+   bits/math-finite.h bits/math-vector.h libm-simd-decl-stubs.h

 # FPU support code.
 aux := setfpucw fpu_control
diff --git a/math/math.h b/math/math.h
index dc532b7..e56afd5 100644
--- a/math/math.h
+++ b/math/math.h
@@ -27,6 +27,9 @@ 

 __BEGIN_DECLS

+/* Get machine-dependent vector math functions declarations */
+#include <bits/math-vector.h>
+
 /* Get machine-dependent HUGE_VAL value (returned on overflow).
    On all IEEE754 machines, this is +Infinity.  */
 #include <bits/huge_val.h>
@@ -49,6 +52,12 @@  __BEGIN_DECLS
    so we can easily declare each function as both `name' and `__name',
    and can declare the float versions `namef' and `__namef'.  */

+#define __SIMD_DECL(function) __CONCAT (__DECL_SIMD_,function)
+
+#define __MATHCALL_VEC(function,suffix, args) \
+  __SIMD_DECL (__MATH_PRECNAME (function,suffix)) \
+  __MATHCALL (function,suffix, args)
+
 #define __MATHCALL(function,suffix, args) \
   __MATHDECL (_Mdouble_,function,suffix, args)
 #define __MATHDECL(type, function,suffix, args) \