[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 Iain Sandoe changed: What|Removed |Added Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #12 from Iain Sandoe --- fixed on open branches
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 Jakub Jelinek changed: What|Removed |Added Target Milestone|8.4 |8.5 --- Comment #11 from Jakub Jelinek --- GCC 8.4.0 has been released, adjusting target milestone.
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 --- Comment #10 from Iain Sandoe --- Author: iains Date: Fri Sep 20 18:29:16 2019 New Revision: 276003 URL: https://gcc.gnu.org/viewcvs?rev=276003=gcc=rev Log: [Darwin, X86, testsuite] Fix PR81058. The tests fail because Darwin indirects common accesses which causes different codegen and the mismatch in output. Placing the vars in regular .data section fixes that. gcc/testsuite/ 2019-09-20 Iain Sandoe Backport from mainline. 2019-05-11 Iain Sandoe PR testsuite/81058 * gcc.target/i386/avx512bw-vpmovswb-1.c: Use regular data section for variables on Darwin, rather than common. * gcc.target/i386/avx512bw-vpmovuswb-1.c: Likewise. * gcc.target/i386/avx512bw-vpmovwb-1.c: Likewise. Modified: branches/gcc-8-branch/gcc/testsuite/ChangeLog branches/gcc-8-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c branches/gcc-8-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c branches/gcc-8-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-1.c
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 Iain Sandoe changed: What|Removed |Added Target Milestone|--- |8.4
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 --- Comment #9 from Iain Sandoe --- Author: iains Date: Tue May 14 17:41:36 2019 New Revision: 271186 URL: https://gcc.gnu.org/viewcvs?rev=271186=gcc=rev Log: darwin, testsuite, backport fixes for PR 81058 2019-05-14 Iain Sandoe Backport from mainline. 2019-05-11 Iain Sandoe PR testsuite/81058 * gcc.target/i386/avx512bw-vpmovswb-1.c: Use regular data section for variables on Darwin, rather than common. * gcc.target/i386/avx512bw-vpmovuswb-1.c: Likewise. * gcc.target/i386/avx512bw-vpmovwb-1.c: Likewise. Modified: branches/gcc-9-branch/gcc/testsuite/ChangeLog branches/gcc-9-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c branches/gcc-9-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c branches/gcc-9-branch/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-1.c
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 --- Comment #8 from Iain Sandoe --- Author: iains Date: Sat May 11 15:05:58 2019 New Revision: 271097 URL: https://gcc.gnu.org/viewcvs?rev=271097=gcc=rev Log: testsuite, darwin] Fix PR81058. The tests fail because Darwin indirects common accesses which causes different codegen and the mismatch in output. Placing the vars in regular .data section fixes that. gcc/testsuite/ 2019-05-11 Iain Sandoe PR testsuite/81058 * gcc.target/i386/avx512bw-vpmovswb-1.c: Use regular data section for variables on Darwin, rather than common. * gcc.target/i386/avx512bw-vpmovuswb-1.c: Likewise. * gcc.target/i386/avx512bw-vpmovwb-1.c: Likewise. Modified: trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c trunk/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c trunk/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-1.c
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 Iain Sandoe changed: What|Removed |Added Attachment #46210|0 |1 is obsolete|| --- Comment #7 from Iain Sandoe --- Created attachment 46215 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46215=edit Revised patch On Dominique's prompting, I had a look at alternatives to skipping these tests. It turns out that we can fix the problem by avoiding the PIC indirections that Darwin's ABI mandates for common accesses. Modulo missing ChangeLog this would be the proposed patch.
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058
--- Comment #6 from Iain Sandoe ---
(In reply to Iain Sandoe from comment #5)
> Created attachment 46210 [details]
> Patch for testing
>
> Darwin doesn't support non-pic for user-space programs, so these tests fail
> (and the codeine is sufficiently different that it would be better to write
> a second set of tests if that is seen as useful). {aside, why is there no
> PIC test for Linux, I guess?]
>
> There is "dynamic-no-pic" for m32 but it doesn't seem worth jumping through
> hoops to make a test case for that, since it's not realistically going to be
> used on a machine supporting AVX512.
there could be a better solution here - remove the need for indirection [Darwin
indirects common accesses] (and hence the change in codegen).
/* { dg-additional-options "-fno-common" { target *-*-darwin* } } */
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058
--- Comment #5 from Iain Sandoe ---
Created attachment 46210
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46210=edit
Patch for testing
Darwin doesn't support non-pic for user-space programs, so these tests fail
(and the codeine is sufficiently different that it would be better to write a
second set of tests if that is seen as useful). {aside, why is there no PIC
test for Linux, I guess?]
There is "dynamic-no-pic" for m32 but it doesn't seem worth jumping through
hoops to make a test case for that, since it's not realistically going to be
used on a machine supporting AVX512.
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 --- Comment #4 from Kirill Yukhin --- Confirmed.
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058
--- Comment #3 from Dominique d'Humieres ---
The problem affects also gcc.target/i386/avx512bw-vpmovwb-1.c. Better patch
--- ../_clean/gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c
2017-06-08 15:14:40.0 +0200
+++ gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c 2017-06-28
15:31:44.0 +0200
@@ -3,15 +3,15 @@
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*res1\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*res1\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*res2\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*res2\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*res3\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*res3\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
#include
vpmovuswb-1.c gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c
--- ../_clean/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c
2017-06-08 15:14:40.0 +0200
+++ gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c2017-06-28
15:33:09.0 +0200
@@ -3,15 +3,15 @@
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*res1\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*res1\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*res2\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%ymm\[0-9\]+\[^\n\]*res2\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*%ymm\[0-9\]+\{%k\[1-7\]\}{z}(?:\n|\[
\\t\]+#)" 1 } } */
-/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*res3\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
+/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%zmm\[0-9\]+\[^\n\]*res3\[^\n\]*\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 { target nonpic } } } */
#include
vpmovwb-1.c
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058
--- Comment #2 from Dominique d'Humieres ---
The following patch fixes the failures
--- ../_clean/gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c
2017-06-08 15:14:40.0 +0200
+++ gcc/testsuite/gcc.target/i386/avx512bw-vpmovswb-1.c 2017-06-26
16:45:03.0 +0200
@@ -1,4 +1,4 @@
-/* { dg-do compile } */
+/* { dg-do compile { target nonpic } } */
/* { dg-options "-mavx512bw -mavx512vl -O2" } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
--- ../_clean/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c
2017-06-08 15:14:40.0 +0200
+++ gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-1.c2017-06-26
16:44:47.0 +0200
@@ -1,4 +1,4 @@
-/* { dg-do compile } */
+/* { dg-do compile { target nonpic } } */
/* { dg-options "-mavx512bw -mavx512vl -O2" } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+(?:\n|\[ \\t\]+#)" 1 } } */
/* { dg-final { scan-assembler-times "vpmovuswb\[
\\t\]+\[^\{\n\]*%xmm\[0-9\]+\[^\n\]*%xmm\[0-9\]+\{%k\[1-7\]\}(?:\n|\[ \\t\]+#)"
1 } } */
[Bug testsuite/81058] FAIL: gcc.target/i386/avx512bw-vpmovu?swb-1.c scan-assembler-times vpmovu?swb.*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81058 Dominique d'Humieres changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2017-06-26 Ever confirmed|0 |1 --- Comment #1 from Dominique d'Humieres --- Any chance to have these failures fixed in a near future?
