[PATCH] tetstsuite gcc.target/i386/ avx512*

[PATCH] tetstsuite gcc.target/i386/ avx512*

[Andreas Tobler]

[-- Attachment #1: Type: text/plain, Size: 973 bytes --]

Hi all,

the attached patch makes it possible to compile and run the failing 
avx512* testcases on FreeBSD (amd64).

See the failing testcases:
https://gcc.gnu.org/ml/gcc-testresults/2015-04/msg02272.html

The header values.h mentions that this interface is obsolete and one 
should use <limits.h> and/or <float.h> instead.

FreeBSD lacks the "values.h" header.

Done so and tested on FreeBSD amd64-unknown-freebsd11.0 and CentOS7.1.

Ok for trunk?

Thanks,
Andreas

2015-04-19  Andreas Tobler  <andreast@gcc.gnu.org>

	* gcc.target/i386/avx512bw-vpermi2w-2.c: Fix includes to use actual
	headers.
	* gcc.target/i386/avx512bw-vpermt2w-2.c: Likewise.
	* gcc.target/i386/avx512bw-vpmaddubsw-2.c: Likewise.
	* gcc.target/i386/avx512bw-vpmaddwd-2.c: Likewise.
	* gcc.target/i386/avx512dq-vfpclasspd-2.c: Likewise.
	* gcc.target/i386/avx512dq-vfpclassps-2.c: Likewise.
	* gcc.target/i386/avx512vbmi-vpermi2b-2.c: Likewise.
	* gcc.target/i386/avx512vbmi-vpermt2b-2.c: Likewise.


[-- Attachment #2: avx512.diff --]
[-- Type: text/plain, Size: 3184 bytes --]

Index: avx512bw-vpermi2w-2.c
===================================================================
--- avx512bw-vpermi2w-2.c	(revision 222220)
+++ avx512bw-vpermi2w-2.c	(working copy)
@@ -6,8 +6,9 @@
 #include "avx512f-helper.h"
 
 #define SIZE (AVX512F_LEN / 16)
-#include "math.h"
-#include "values.h"
+#include <math.h>
+#include <limits.h>
+#include <float.h>
 #include "avx512f-mask-type.h"
 
 #define NUM 32
Index: avx512bw-vpermt2w-2.c
===================================================================
--- avx512bw-vpermt2w-2.c	(revision 222220)
+++ avx512bw-vpermt2w-2.c	(working copy)
@@ -6,8 +6,9 @@
 #include "avx512f-helper.h"
 
 #define SIZE (AVX512F_LEN / 16)
-#include "math.h"
-#include "values.h"
+#include <math.h>
+#include <limits.h>
+#include <float.h>
 #include "avx512f-mask-type.h"
 
 #define NUM 32
Index: avx512bw-vpmaddubsw-2.c
===================================================================
--- avx512bw-vpmaddubsw-2.c	(revision 222220)
+++ avx512bw-vpmaddubsw-2.c	(working copy)
@@ -6,7 +6,8 @@
 #include "avx512f-helper.h"
 
 #include <math.h>
-#include <values.h>
+#include <limits.h>
+#include <float.h>
 #define SIZE (AVX512F_LEN / 16)
 #include "avx512f-mask-type.h"
 
Index: avx512bw-vpmaddwd-2.c
===================================================================
--- avx512bw-vpmaddwd-2.c	(revision 222220)
+++ avx512bw-vpmaddwd-2.c	(working copy)
@@ -6,7 +6,8 @@
 #include "avx512f-helper.h"
 
 #include <math.h>
-#include <values.h>
+#include <limits.h>
+#include <float.h>
 #define SIZE (AVX512F_LEN / 32)
 #include "avx512f-mask-type.h"
 
Index: avx512dq-vfpclasspd-2.c
===================================================================
--- avx512dq-vfpclasspd-2.c	(revision 222220)
+++ avx512dq-vfpclasspd-2.c	(working copy)
@@ -6,7 +6,8 @@
 #include "avx512f-helper.h"
 
 #include <math.h>
-#include <values.h>
+#include <limits.h>
+#include <float.h>
 #define SIZE (AVX512F_LEN / 64)
 #include "avx512f-mask-type.h"
 
Index: avx512dq-vfpclassps-2.c
===================================================================
--- avx512dq-vfpclassps-2.c	(revision 222220)
+++ avx512dq-vfpclassps-2.c	(working copy)
@@ -6,7 +6,8 @@
 #include "avx512f-helper.h"
 
 #include <math.h>
-#include <values.h>
+#include <limits.h>
+#include <float.h>
 #include "avx512f-mask-type.h"
 #define SIZE (AVX512F_LEN / 32)
 
Index: avx512vbmi-vpermi2b-2.c
===================================================================
--- avx512vbmi-vpermi2b-2.c	(revision 222220)
+++ avx512vbmi-vpermi2b-2.c	(working copy)
@@ -7,8 +7,9 @@
 #include "avx512f-helper.h"
 
 #define SIZE (AVX512F_LEN / 8)
-#include "math.h"
-#include "values.h"
+#include <math.h>
+#include <limits.h>
+#include <float.h>
 #include "avx512f-mask-type.h"
 
 #define NUM 32
Index: avx512vbmi-vpermt2b-2.c
===================================================================
--- avx512vbmi-vpermt2b-2.c	(revision 222220)
+++ avx512vbmi-vpermt2b-2.c	(working copy)
@@ -7,8 +7,9 @@
 #include "avx512f-helper.h"
 
 #define SIZE (AVX512F_LEN / 8)
-#include "math.h"
-#include "values.h"
+#include <math.h>
+#include <limits.h>
+#include <float.h>
 #include "avx512f-mask-type.h"
 
 #define NUM 32

Re: [PATCH] tetstsuite gcc.target/i386/ avx512*

[Kirill Yukhin]

Hello Andreas,
On 19 Apr 21:56, Andreas Tobler wrote:
> Done so and tested on FreeBSD amd64-unknown-freebsd11.0 and CentOS7.1.
> 
> Ok for trunk?
The patch is OK for trunk and for gcc-5 branch (when it is open).

Thanks for fixing this!

--
K

Re: [PATCH] tetstsuite gcc.target/i386/ avx512*

[Andreas Tobler]

Hi Kirill,

On 21.04.15 10:28, Kirill Yukhin wrote:

> On 19 Apr 21:56, Andreas Tobler wrote:
>> Done so and tested on FreeBSD amd64-unknown-freebsd11.0 and CentOS7.1.
>>
>> Ok for trunk?
> The patch is OK for trunk and for gcc-5 branch (when it is open).
>
> Thanks for fixing this!

Done on trunk and gcc-5.

Thanks for the review!

Andreas