This is the mail archive of the
glibc-bugs@sourceware.org
mailing list for the glibc project.
[Bug libc/20689] Test for FMA should also check for AVX.
- From: "cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>
- To: glibc-bugs at sourceware dot org
- Date: Mon, 17 Oct 2016 23:40:57 +0000
- Subject: [Bug libc/20689] Test for FMA should also check for AVX.
- Auto-submitted: auto-generated
- References: <bug-20689-131@http.sourceware.org/bugzilla/>
https://sourceware.org/bugzilla/show_bug.cgi?id=20689
--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via b3d17c1cf29ac8cd6b2fcd1db7d824e4af9ea505 (commit)
from 6f322a894704e7959b9cee7c2cc7ec1963a26e73 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=b3d17c1cf29ac8cd6b2fcd1db7d824e4af9ea505
commit b3d17c1cf29ac8cd6b2fcd1db7d824e4af9ea505
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Mon Oct 17 19:35:34 2016 -0400
Bug 20689: Fix FMA and AVX2 detection on Intel
In the Intel Architecture Instruction Set Extensions Programming
reference the recommended way to test for FMA in section
'2.2.1 Detection of FMA' is:
"Application Software must identify that hardware supports AVX as
explained in ... after that it must also detect support for FMA..."
We don't do that in glibc. We use osxsave to detect the use of xgetbv,
and after that we check for AVX and FMA orthogonally. It is conceivable
that you could have the AVX bit clear and the FMA bit in an undefined
state.
This commit fixes FMA and AVX2 detection to depend on usable AVX
as required by the recommended Intel sequences.
v1: https://www.sourceware.org/ml/libc-alpha/2016-10/msg00241.html
v2: https://www.sourceware.org/ml/libc-alpha/2016-10/msg00265.html
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 6 ++++++
sysdeps/x86/cpu-features.c | 24 ++++++++++++++----------
2 files changed, 20 insertions(+), 10 deletions(-)
--
You are receiving this mail because:
You are on the CC list for the bug.