altivec.md (altivec_lvsl): New define_expand.
authorBill Schmidt <wschmidt@linux.vnet.ibm.com>
Fri, 3 Oct 2014 22:38:39 +0000 (22:38 +0000)
committerWilliam Schmidt <wschmidt@gcc.gnu.org>
Fri, 3 Oct 2014 22:38:39 +0000 (22:38 +0000)
[gcc]

2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

* altivec.md (altivec_lvsl): New define_expand.
(altivec_lvsl_direct): Rename define_insn from altivec_lvsl.
(altivec_lvsr): New define_expand.
(altivec_lvsr_direct): Rename define_insn from altivec_lvsr.
* rs6000.c (rs6000_expand_builtin): Change to use
altivec_lvs[lr]_direct; remove commented-out code.

[gcc/testsuite]

2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

* gcc.target/powerpc/lvsl-lvsr.c: New test.

From-SVN: r215882

gcc/ChangeLog
gcc/config/rs6000/altivec.md
gcc/config/rs6000/rs6000.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/powerpc/lvsl-lvsr.c [new file with mode: 0644]

index 23f3d23..f7724f5 100644 (file)
@@ -1,5 +1,14 @@
 2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
 
+       * altivec.md (altivec_lvsl): New define_expand.
+       (altivec_lvsl_direct): Rename define_insn from altivec_lvsl.
+       (altivec_lvsr): New define_expand.
+       (altivec_lvsr_direct): Rename define_insn from altivec_lvsr.
+       * rs6000.c (rs6000_expand_builtin): Change to use
+       altivec_lvs[lr]_direct; remove commented-out code.
+
+2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
+
        * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin):
        Issue a warning message when vec_lvsl or vec_lvsr is used with a
        little endian target.
index 459179a..02ea142 100644 (file)
   "dststt %0,%1,%2"
   [(set_attr "type" "vecsimple")])
 
-(define_insn "altivec_lvsl"
+(define_expand "altivec_lvsl"
+  [(use (match_operand:V16QI 0 "register_operand" ""))
+   (use (match_operand:V16QI 1 "memory_operand" ""))]
+  "TARGET_ALTIVEC"
+{
+  if (VECTOR_ELT_ORDER_BIG)
+    emit_insn (gen_altivec_lvsl_direct (operands[0], operands[1]));
+  else
+    {
+      int i;
+      rtx mask, perm[16], constv, vperm;
+      mask = gen_reg_rtx (V16QImode);
+      emit_insn (gen_altivec_lvsl_direct (mask, operands[1]));
+      for (i = 0; i < 16; ++i)
+        perm[i] = GEN_INT (i);
+      constv = gen_rtx_CONST_VECTOR (V16QImode, gen_rtvec_v (16, perm));
+      constv = force_reg (V16QImode, constv);
+      vperm = gen_rtx_UNSPEC (V16QImode, gen_rtvec (3, mask, mask, constv),
+                              UNSPEC_VPERM);
+      emit_insn (gen_rtx_SET (VOIDmode, operands[0], vperm));
+    }
+  DONE;
+})
+
+(define_insn "altivec_lvsl_direct"
   [(set (match_operand:V16QI 0 "register_operand" "=v")
        (unspec:V16QI [(match_operand:V16QI 1 "memory_operand" "Z")]
                      UNSPEC_LVSL))]
   "lvsl %0,%y1"
   [(set_attr "type" "vecload")])
 
-(define_insn "altivec_lvsr"
+(define_expand "altivec_lvsr"
+  [(use (match_operand:V16QI 0 "register_operand" ""))
+   (use (match_operand:V16QI 1 "memory_operand" ""))]
+  "TARGET_ALTIVEC"
+{
+  if (VECTOR_ELT_ORDER_BIG)
+    emit_insn (gen_altivec_lvsr_direct (operands[0], operands[1]));
+  else
+    {
+      int i;
+      rtx mask, perm[16], constv, vperm;
+      mask = gen_reg_rtx (V16QImode);
+      emit_insn (gen_altivec_lvsr_direct (mask, operands[1]));
+      for (i = 0; i < 16; ++i)
+        perm[i] = GEN_INT (i);
+      constv = gen_rtx_CONST_VECTOR (V16QImode, gen_rtvec_v (16, perm));
+      constv = force_reg (V16QImode, constv);
+      vperm = gen_rtx_UNSPEC (V16QImode, gen_rtvec (3, mask, mask, constv),
+                              UNSPEC_VPERM);
+      emit_insn (gen_rtx_SET (VOIDmode, operands[0], vperm));
+    }
+  DONE;
+})
+
+(define_insn "altivec_lvsr_direct"
   [(set (match_operand:V16QI 0 "register_operand" "=v")
        (unspec:V16QI [(match_operand:V16QI 1 "memory_operand" "Z")]
                      UNSPEC_LVSR))]
index 16847aa..489c65e 100644 (file)
@@ -13898,8 +13898,8 @@ rs6000_expand_builtin (tree exp, rtx target, rtx subtarget ATTRIBUTE_UNUSED,
     case ALTIVEC_BUILTIN_MASK_FOR_LOAD:
     case ALTIVEC_BUILTIN_MASK_FOR_STORE:
       {
-       int icode = (BYTES_BIG_ENDIAN ? (int) CODE_FOR_altivec_lvsr
-                    : (int) CODE_FOR_altivec_lvsl);
+       int icode = (BYTES_BIG_ENDIAN ? (int) CODE_FOR_altivec_lvsr_direct
+                    : (int) CODE_FOR_altivec_lvsl_direct);
        enum machine_mode tmode = insn_data[icode].operand[0].mode;
        enum machine_mode mode = insn_data[icode].operand[1].mode;
        tree arg;
@@ -13927,7 +13927,6 @@ rs6000_expand_builtin (tree exp, rtx target, rtx subtarget ATTRIBUTE_UNUSED,
            || ! (*insn_data[icode].operand[0].predicate) (target, tmode))
          target = gen_reg_rtx (tmode);
 
-       /*pat = gen_altivec_lvsr (target, op);*/
        pat = GEN_FCN (icode) (target, op);
        if (!pat)
          return 0;
index 489f3ee..036b171 100644 (file)
@@ -1,5 +1,9 @@
 2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
 
+       * gcc.target/powerpc/lvsl-lvsr.c: New test.
+
+2014-10-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
+
        * g++.dg/ext/altivec-2.C: Compile with -Wno-deprecated to avoid
        failing with the new warning message.
        * gcc.dg/vmx/3c-01a.c: Likewise.
diff --git a/gcc/testsuite/gcc.target/powerpc/lvsl-lvsr.c b/gcc/testsuite/gcc.target/powerpc/lvsl-lvsr.c
new file mode 100644 (file)
index 0000000..71dd0a2
--- /dev/null
@@ -0,0 +1,21 @@
+/* Test expected code generation for lvsl and lvsr on little endian.
+   Note that lvsl and lvsr are each produced once, but the filename
+   causes them to appear twice in the file.  */
+
+/* { dg-do compile { target { powerpc64le-*-* } } } */
+/* { dg-options "-O0 -Wno-deprecated" } */
+/* { dg-final { scan-assembler-times "lvsl" 2 } } */
+/* { dg-final { scan-assembler-times "lvsr" 2 } } */
+/* { dg-final { scan-assembler-times "lxvd2x" 2 } } */
+/* { dg-final { scan-assembler-times "vperm" 2 } } */
+
+
+#include <altivec.h>
+
+float f[20];
+
+void foo ()
+{
+  vector unsigned char a = vec_lvsl (4, f);
+  vector unsigned char b = vec_lvsr (8, f);
+}