Re: patch for PR84359

2018-02-14 Thread Uros Bizjak
>  The following patch fixes
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84359
>
>  Committed as rev. 257628.
>
>
> Index: testsuite/gcc.target/i386/pr57193.c
> ===
> --- testsuite/gcc.target/i386/pr57193.c (revision 257537)
> +++ testsuite/gcc.target/i386/pr57193.c (working copy)
> @@ -1,5 +1,5 @@
>  /* { dg-do compile { target { ! ia32 } } } */
> -/* { dg-options "-O2" } */
> +/* { dg-options "-O2 -march=x86-64" } */
>  /* { dg-final { scan-assembler-times "movdqa" 2 } } */

The preferred way to stabilize assembler dumps on testcases, which
depend on -march is to manually specify -mno-sseX, as in the attached
patch.

Also, the test will work on 32bit x86 targets with -msse2, so the
patch also removes target selector from the testcase.

Tested on x86_64-linux-gnu {,-m32}, will be committed to mainline ASAP.

Uros.

Index: gcc.target/i386/pr57193.c
===
--- gcc.target/i386/pr57193.c(revision 257659)
+++ gcc.target/i386/pr57193.c(working copy)
@@ -1,5 +1,5 @@
-/* { dg-do compile { target { ! ia32 } } } */
-/* { dg-options "-O2 -march=x86-64" } */
+/* { dg-do compile } */
+/* { dg-options "-O2 -msse2 -mno-sse3" } */
 /* { dg-final { scan-assembler-times "movdqa" 2 } } */

 #include 


patch for PR84359

2018-02-13 Thread Vladimir Makarov

  The following patch fixes

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84359

  Committed as rev. 257628.

Index: testsuite/ChangeLog
===
--- testsuite/ChangeLog	(revision 257627)
+++ testsuite/ChangeLog	(working copy)
@@ -1,3 +1,8 @@
+2018-02-13  Vladimir Makarov  
+
+	PR target/84359
+	* gcc.target/i386/57193.c: Add -march=x86-64.
+
 2018-02-13  Paolo Bonzini  
 
 	PR sanitizer/84340
Index: testsuite/gcc.target/i386/pr57193.c
===
--- testsuite/gcc.target/i386/pr57193.c	(revision 257537)
+++ testsuite/gcc.target/i386/pr57193.c	(working copy)
@@ -1,5 +1,5 @@
 /* { dg-do compile { target { ! ia32 } } } */
-/* { dg-options "-O2" } */
+/* { dg-options "-O2 -march=x86-64" } */
 /* { dg-final { scan-assembler-times "movdqa" 2 } } */
 
 #include