public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Re: [PATCH] testsuite/63175 - fix powerpc64 test failure
@ 2015-03-23 17:11 David Edelsohn
  0 siblings, 0 replies; 3+ messages in thread
From: David Edelsohn @ 2015-03-23 17:11 UTC (permalink / raw)
  To: Martin Sebor; +Cc: GCC Patches

The patch below fixes a failure of the test on powerpc64.
The test looks for the lxv (Load VSX Vector) instruction
which is emitted on powerpc64le but on powerpc64 gcc emits
an lvx (Load Vector Indexed). Both are correct here.

Is this okay to commit to trunk?

+       PR testsuite/63175
+       * gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c: Scan
+       assembly for lvx in addition to lxv.

Okay.

Thanks, David

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] testsuite/63175 - fix powerpc64 test failure
  2015-03-23 16:56 Martin Sebor
@ 2015-03-23 17:13 ` Andreas Schwab
  0 siblings, 0 replies; 3+ messages in thread
From: Andreas Schwab @ 2015-03-23 17:13 UTC (permalink / raw)
  To: Martin Sebor; +Cc: Gcc Patch List

Martin Sebor <msebor@redhat.com> writes:

> -/* { dg-final { scan-assembler "\t\(lxv|lvsr|stxv\)" } } */
> +/* { dg-final { scan-assembler "\t\(lvx|lxv|lvsr|stxv\)" } } */

You might want to remove the backslashes before the parens, which are
ignored anyway.

Andreas.

-- 
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] testsuite/63175 - fix powerpc64 test failure
@ 2015-03-23 16:56 Martin Sebor
  2015-03-23 17:13 ` Andreas Schwab
  0 siblings, 1 reply; 3+ messages in thread
From: Martin Sebor @ 2015-03-23 16:56 UTC (permalink / raw)
  To: Gcc Patch List

The patch below fixes a failure of the test on powerpc64.
The test looks for the lxv (Load VSX Vector) instruction
which is emitted on powerpc64le but on powerpc64 gcc emits
an lvx (Load Vector Indexed). Both are correct here.

Is this okay to commit to trunk?

Martin

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 884fb62..d781c02 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2015-03-23  Martin Sebor  <msebor@redhat.com>
+
+       PR testsuite/63175
+       * gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c: Scan
+       assembly for lvx in addition to lxv.
+
  2015-03-20  Richard Biener  <rguenther@suse.de>

         PR middle-end/64715
diff --git 
a/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c 
b/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c
index 73c0afa..9f3c89c 100644
--- a/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c
+++ b/gcc/testsuite/gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c
@@ -25,6 +25,6 @@ main1 (void)
     with no word loads (lw, lwu, lwz, lwzu, or their indexed forms)
     or word stores (stw, stwu, stwx, stwux, or their indexed forms).  */

-/* { dg-final { scan-assembler "\t\(lxv|lvsr|stxv\)" } } */
+/* { dg-final { scan-assembler "\t\(lvx|lxv|lvsr|stxv\)" } } */
  /* { dg-final { scan-assembler-not "\tlwz?u?x? " } } */
  /* { dg-final { scan-assembler-not "\tstwu?x? " } } */

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-03-23 17:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-23 17:11 [PATCH] testsuite/63175 - fix powerpc64 test failure David Edelsohn
  -- strict thread matches above, loose matches on Subject: below --
2015-03-23 16:56 Martin Sebor
2015-03-23 17:13 ` Andreas Schwab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).