public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [COMMITTED PATCH] Fix tst-audit10 build when -mavx512f is not supported.
@ 2016-03-08 20:36 Roland McGrath
  2016-03-10 11:16 ` Florian Weimer
  0 siblings, 1 reply; 2+ messages in thread
From: Roland McGrath @ 2016-03-08 20:36 UTC (permalink / raw)
  To: GNU C. Library; +Cc: Florian Weimer

This unbroke my local build with the same compiler the bot uses, and
so I expect it to unbreak the bot too.


Thanks,
Roland


2016-03-08  Roland McGrath  <roland@hack.frob.com>

	* sysdeps/x86_64/tst-audit10.c: #include <cpu-features.h>.
	* sysdeps/x86_64/tst-audit10-aux.c: Move audit_test extern decl ...
	(tst_audit10_aux) [__AVX512F__]: ... here.

diff --git a/sysdeps/x86_64/tst-audit10-aux.c b/sysdeps/x86_64/tst-audit10-aux.c
index 4398b8f..992a16c 100644
--- a/sysdeps/x86_64/tst-audit10-aux.c
+++ b/sysdeps/x86_64/tst-audit10-aux.c
@@ -20,13 +20,13 @@
 #include <stdlib.h>
 #include <string.h>
 
-extern __m512i audit_test (__m512i, __m512i, __m512i, __m512i,
-			   __m512i, __m512i, __m512i, __m512i);
-
 int
 tst_audit10_aux (void)
 {
 #ifdef __AVX512F__
+  extern __m512i audit_test (__m512i, __m512i, __m512i, __m512i,
+                             __m512i, __m512i, __m512i, __m512i);
+
   __m512i zmm = _mm512_setzero_si512 ();
   __m512i ret = audit_test (zmm, zmm, zmm, zmm, zmm, zmm, zmm, zmm);
 
diff --git a/sysdeps/x86_64/tst-audit10.c b/sysdeps/x86_64/tst-audit10.c
index 92e0cb4..a487b40 100644
--- a/sysdeps/x86_64/tst-audit10.c
+++ b/sysdeps/x86_64/tst-audit10.c
@@ -17,6 +17,7 @@
    <http://www.gnu.org/licenses/>.  */
 
 #include <cpuid.h>
+#include <cpu-features.h>
 
 int tst_audit10_aux (void);
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [COMMITTED PATCH] Fix tst-audit10 build when -mavx512f is not supported.
  2016-03-08 20:36 [COMMITTED PATCH] Fix tst-audit10 build when -mavx512f is not supported Roland McGrath
@ 2016-03-10 11:16 ` Florian Weimer
  0 siblings, 0 replies; 2+ messages in thread
From: Florian Weimer @ 2016-03-10 11:16 UTC (permalink / raw)
  To: Roland McGrath; +Cc: GNU C. Library

On 03/08/2016 09:36 PM, Roland McGrath wrote:
> This unbroke my local build with the same compiler the bot uses, and
> so I expect it to unbreak the bot too.

Sorry, and thanks for correcting this.  I verified that it fixes things
for GCC 4.7 as well.

Florian

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-10 11:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-08 20:36 [COMMITTED PATCH] Fix tst-audit10 build when -mavx512f is not supported Roland McGrath
2016-03-10 11:16 ` Florian Weimer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).