diff mbox

[1/14,x86_64] Vector math functions (vector cos)

Message ID CAMXFM3tFo1KU3y-55zzWVdT9GAG_ND15_ZwGNkZQVp2TCoBgJA@mail.gmail.com
State New
Headers show

Commit Message

Andrew Senkevich June 8, 2015, 12:46 p.m. UTC 
2015-06-05 19:44 GMT+03:00 Joseph Myers <joseph@codesourcery.com>:
> Note that the addition of the first libmvec functions should be
> accompanied by a NEWS entry describing this new feature in 2.22.  (That
> NEWS entry can then be updated for each new function added - of course
> anything added after 2.22 is released gets a separate NEWS entry for
> 2.23.)

Is this update ok?


I also would like to add wiki page (with content mostly based on
https://sourceware.org/glibc/wiki/libm#Addition_of_x86_64_vector_math_functions_to_Glibc)
to 5.4. Internals Documentation at
https://sourceware.org/glibc/wiki#Development and add according link
to NEWS.


--
WBR,
Andrew

Comments

Joseph Myers June 8, 2015, 1:06 p.m. UTC | #1 
On Mon, 8 Jun 2015, Andrew Senkevich wrote:

> +* Added vector math library named libmvec with the following vectorized x86_64
> +  implementations: cos.
> +  Naming of x86_64 implementations is in accordance with
> +  Vector ABI <https://groups.google.com/forum/#!topic/x86-64-abi/LmppCfN1rZ4>.
> +  Vector math library build and ABI testing enabled by default for x86_64.

I don't think the NEWS file should point to a site requiring non-free 
JavaScript; host a copy of the ABI somewhere else (maybe on the glibc 
wiki).

I think the NEWS file needs to give information from the perspective of 
people building and using glibc.  That includes: the library can be 
disabled with --disable-mathvec (the configure option also needs 
documenting in manual/install.texi, with INSTALL being regenerated); use 
of the functions is enabled with -fopenmp -ffast-math -O3 (or whatever) 
and the library is linked in as needed when using -lm rather than needing 
to specify -lmvec explicitly.
diff mbox

Patch

diff --git a/NEWS b/NEWS
index 881e61c..18ce8d5 100644
--- a/NEWS
+++ b/NEWS
@@ -50,6 +50,12 @@  Version 2.22
 * CVE-2014-8121 The NSS backends shared internal state between the getXXent
   and getXXbyYY NSS calls for the same database, causing a denial-of-service
   condition in some applications.
+
+* Added vector math library named libmvec with the following vectorized x86_64
+  implementations: cos.
+  Naming of x86_64 implementations is in accordance with
+  Vector ABI <https://groups.google.com/forum/#!topic/x86-64-abi/LmppCfN1rZ4>.
+  Vector math library build and ABI testing enabled by default for x86_64.
 ^L
 Version 2.21