diff mbox

ix86: fix PR/65105 testcase 2

Message ID 577643C002000078000FA50E@prv-mh.provo.novell.com
State New
Headers show

Commit Message

Jan Beulich July 1, 2016, 8:19 a.m. UTC 
I cannot see how without allowing the compiler to use SSE2 instructions
(as is done by all other tests for this PR scanning for particular
instructions) this test could ever have succeeded anywhere.

gcc/testsuite/
2016-07-01  Jan Beulich  <jbeulich@suse.com>

	* gcc.target/i386/pr65105-2.c: Add -msse2.

Comments

Uros Bizjak July 1, 2016, 8:35 a.m. UTC | #1 
On Fri, Jul 1, 2016 at 10:19 AM, Jan Beulich <JBeulich@suse.com> wrote:
> I cannot see how without allowing the compiler to use SSE2 instructions
> (as is done by all other tests for this PR scanning for particular
> instructions) this test could ever have succeeded anywhere.

Well, most of 32bit target testing nowadays is done on x86_64
multilibs, where SSE2 is also enabled by default on 32bit.

> gcc/testsuite/
> 2016-07-01  Jan Beulich  <jbeulich@suse.com>
>
>         * gcc.target/i386/pr65105-2.c: Add -msse2.

OK everywhere.

Thanks,
Uros.

> --- 2016-06-30/gcc/testsuite/gcc.target/i386/pr65105-2.c
> +++ 2016-06-30/gcc/testsuite/gcc.target/i386/pr65105-2.c
> @@ -1,6 +1,6 @@
>  /* PR target/pr65105 */
>  /* { dg-do compile { target { ia32 } } } */
> -/* { dg-options "-O2" } */
> +/* { dg-options "-O2 -msse2" } */
>  /* { dg-final { scan-assembler "por" } } */
>
>  long long i1, i2, res;
>
>
>
diff mbox

Patch

--- 2016-06-30/gcc/testsuite/gcc.target/i386/pr65105-2.c
+++ 2016-06-30/gcc/testsuite/gcc.target/i386/pr65105-2.c
@@ -1,6 +1,6 @@ 
 /* PR target/pr65105 */
 /* { dg-do compile { target { ia32 } } } */
-/* { dg-options "-O2" } */
+/* { dg-options "-O2 -msse2" } */
 /* { dg-final { scan-assembler "por" } } */
 
 long long i1, i2, res;