diff mbox

How to generate AVX512 instructions now (just to look at them).

Message ID 20140129153317.GA23059@msticlxl57.ims.intel.com
State New
Headers show

Commit Message

Kirill Yukhin Jan. 29, 2014, 3:33 p.m. UTC 
Hello,
I think its time to remove `XPASS' from corresponding tests.
On 03 Jan 22:11, Jakub Jelinek wrote:
> Hi!
> 
> On Fri, Jan 03, 2014 at 08:58:30PM +0100, Toon Moene wrote:
> > I don't doubt that would work, what I'm interested in, is (cat verintlin.f):
> 
> Well, you need gather loads for that and there you hit PR target/59617.

testsuite/
	PR target/59617
	* gcc.target/i386/avx512f-gather-2.c: Remove XPASS
	* gcc.target/i386/avx512f-gather-5.c: Ditto.

Patch in the bottom. Updated tests pass.
Is it ok for trunk?

--
Thanks, K

 gcc/testsuite/gcc.target/i386/avx512f-gather-2.c | 8 ++++----
 gcc/testsuite/gcc.target/i386/avx512f-gather-5.c | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

Comments

Jakub Jelinek Jan. 29, 2014, 3:57 p.m. UTC | #1 
On Wed, Jan 29, 2014 at 06:33:21PM +0300, Kirill Yukhin wrote:
> I think its time to remove `XPASS' from corresponding tests.
> On 03 Jan 22:11, Jakub Jelinek wrote:
> > Hi!
> > 
> > On Fri, Jan 03, 2014 at 08:58:30PM +0100, Toon Moene wrote:
> > > I don't doubt that would work, what I'm interested in, is (cat verintlin.f):
> > 
> > Well, you need gather loads for that and there you hit PR target/59617.
> 
> testsuite/
> 	PR target/59617
> 	* gcc.target/i386/avx512f-gather-2.c: Remove XPASS
> 	* gcc.target/i386/avx512f-gather-5.c: Ditto.
> 
> Patch in the bottom. Updated tests pass.
> Is it ok for trunk?

Ok, thanks.  Sorry for not removing those myself.

	Jakub
diff mbox

Patch

diff --git a/gcc/testsuite/gcc.target/i386/avx512f-gather-2.c b/gcc/testsuite/gcc.target/i386/avx512f-gather-2.c
index 8664192..f20d3db 100644
--- a/gcc/testsuite/gcc.target/i386/avx512f-gather-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx512f-gather-2.c
@@ -3,9 +3,9 @@ 
 
 #include "avx512f-gather-1.c"
 
-/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*ymm" { xfail { *-*-* } } } } */  /* PR59617 */
-/* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*ymm" { xfail { *-*-* } } } } */  /* PR59617 */
-/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*xmm" { xfail { *-*-* } } } } */  /* PR59617 */
-/* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*xmm" { xfail { lp64 } } } } */  /* PR59617 */
+/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*ymm" } } */
+/* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*ymm" } } */
+/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*xmm" } } */
+/* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*xmm" } } */
 /* { dg-final { scan-tree-dump-times "note: vectorized 1 loops in function" 16 "vect" } } */
 /* { dg-final { cleanup-tree-dump "vect" } } */
diff --git a/gcc/testsuite/gcc.target/i386/avx512f-gather-5.c b/gcc/testsuite/gcc.target/i386/avx512f-gather-5.c
index 5edd446..d2237da 100644
--- a/gcc/testsuite/gcc.target/i386/avx512f-gather-5.c
+++ b/gcc/testsuite/gcc.target/i386/avx512f-gather-5.c
@@ -3,8 +3,8 @@ 
 
 #include "avx512f-gather-4.c"
 
-/* { dg-final { scan-assembler "gather\[^\n\]*zmm" { xfail { *-*-* } } } } */ /* PR59617 */
-/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*ymm" { xfail { *-*-* } } } } */ /* PR59617 */
+/* { dg-final { scan-assembler "gather\[^\n\]*zmm" } } */
+/* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*ymm" } } */
 /* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*ymm" } } */
 /* { dg-final { scan-assembler-not "gather\[^\n\]*ymm\[^\n\]*xmm" } } */
 /* { dg-final { scan-assembler-not "gather\[^\n\]*xmm\[^\n\]*xmm" } } */