Recently I committed a patch solving

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

The patch resulted in test vsx-simode2.c failure.  Here is the difference in generated code:

@@ -13,9 +13,8 @@ foo:
 .LFB0:
        .cfi_startproc
        std 3,-16(1)
-       ori 2,2,0
-       lwz 9,-12(1)
-       mtvsrwz 32,9
+       addi 9,1,-12
+       lxsiwzx 32,0,9

The new version is one insn less.  So I propose the following patch changing the expected code generation.

Is it ok to commit it?


Index: ChangeLog
===================================================================
--- ChangeLog   (revision 268581)
+++ ChangeLog   (working copy)
@@ -1,3 +1,8 @@
+2019-02-08  Vladimir Makarov  <vmaka...@redhat.com>
+
+       * gcc.target/powerpc/vsx-simode2.c: Expect lxsiwzx instead of
+       mtvsrwz.
+
 2019-02-06  Richard Biener  <rguent...@suse.de>
 
        PR tree-optimization/89182
Index: gcc.target/powerpc/vsx-simode2.c
===================================================================
--- gcc.target/powerpc/vsx-simode2.c    (revision 268581)
+++ gcc.target/powerpc/vsx-simode2.c    (working copy)
@@ -11,5 +11,5 @@ unsigned int foo (unsigned int u)
   return ret;
 }
 
-/* { dg-final { scan-assembler "mtvsrwz" } } */
+/* { dg-final { scan-assembler "lxsiwzx" } } */
 /* { dg-final { scan-assembler "mfvsrwz" } } */

Reply via email to