re PR target/35574 (unrecognizable insn generated for vector move)

	PR target/35574
	* config/sparc/constraints.md (D): New.
	* config/sparc/predicates.md (const_double_or_vector_operand):
	New.
	* config/sparc/sparc.c (sparc_extra_constraint_check): Handle the
	'D' constraint.
	* config/sparc/sparc.md (*movdf_insn_sp32_v9, *movdf_insn_sp64):
	Use the 'D' constraint in addition to 'F' in some alternatives.
	(DF splitter): Generalize for V64mode.
	* doc/md.texi (SPARC): Document the 'D' constraint.

From-SVN: r141644

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e0f76e8456a335439b7988764fe94c7cd24d62a9..44b888e8537381696c1b6312c7a38fc9aec2eda9 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,16 @@
+2008-11-06  Kazu Hirata  <kazu@codesourcery.com>
+
+       PR target/35574
+       * config/sparc/constraints.md (D): New.
+       * config/sparc/predicates.md (const_double_or_vector_operand):
+       New.
+       * config/sparc/sparc.c (sparc_extra_constraint_check): Handle the
+       'D' constraint.
+       * config/sparc/sparc.md (*movdf_insn_sp32_v9, *movdf_insn_sp64):
+       Use the 'D' constraint in addition to 'F' in some alternatives.
+       (DF splitter): Generalize for V64mode.
+       * doc/md.texi (SPARC): Document the 'D' constraint.
+
 2008-11-06  Uros Bizjak  <ubizjak@gmail.com>
 
        * reg-stack.c (reg_to_stack): Generate +QNaN using real_nan.

diff --git a/gcc/config/sparc/constraints.md b/gcc/config/sparc/constraints.md
index 6d505649c768524aeb7dce5bd27addae14aa7b8a..ee0fbbda69ac84ae272b90950a8efdc32b957b3c 100644 (file)
--- a/gcc/config/sparc/constraints.md
+++ b/gcc/config/sparc/constraints.md
@@ -100,6 +100,11 @@
 ;; Our memory extra constraints have to emulate the behavior of 'm' and 'o',
 ;; i.e. accept pseudo-registers during reload.
 
+(define_constraint "D"
+ "const_vector"
+  (and (match_code "const_vector")
+       (match_test "GET_MODE_CLASS (GET_MODE (op)) == MODE_VECTOR_INT")))
+
 (define_constraint "Q"
  "Floating-point constant that can be loaded with a sethi instruction"
  (and (match_code "const_double")

diff --git a/gcc/config/sparc/predicates.md b/gcc/config/sparc/predicates.md
index fb6f98be3721d15f8c799cd5d29f558cf5426bd7..379c73c8f82509f0d3d7d3c8405b92f23f865e21 100644 (file)
--- a/gcc/config/sparc/predicates.md
+++ b/gcc/config/sparc/predicates.md
@@ -83,6 +83,10 @@
   return fp_high_losum_p (op);
 })
 
+;; Return true if OP is a const_double or const_vector.
+(define_predicate "const_double_or_vector_operand"
+  (match_code "const_double,const_vector"))
+
 
 ;; Predicates for symbolic constants.
 

diff --git a/gcc/config/sparc/sparc.md b/gcc/config/sparc/sparc.md
index dc6c3a31645be1757ebf9ac8fdf632fb1969e2cb..7dadde2b443cb1eb51bee4498e5f31ca2292a600 100644 (file)
--- a/gcc/config/sparc/sparc.md
+++ b/gcc/config/sparc/sparc.md
@@ -2571,7 +2571,7 @@
 ;; We have available v9 double floats but not 64-bit integer registers.
 (define_insn "*movdf_insn_sp32_v9"
   [(set (match_operand:V64 0 "nonimmediate_operand" "=b,e,e,T,W,U,T,f,*r,o")
-        (match_operand:V64 1 "input_operand" "GY,e,W#F,GY,e,T,U,o#F,*roGYF,*rGYf"))]
+        (match_operand:V64 1 "input_operand" "GY,e,W#F,GY,e,T,U,o#F,*roGYDF,*rGYf"))]
   "TARGET_FPU
    && TARGET_V9
    && ! TARGET_ARCH64
@@ -2612,7 +2612,7 @@
 ;; We have available both v9 double floats and 64-bit integer registers.
 (define_insn "*movdf_insn_sp64"
   [(set (match_operand:V64 0 "nonimmediate_operand" "=b,e,e,W,*r,*r,m,*r")
-        (match_operand:V64 1 "input_operand"    "GY,e,W#F,e,*rGY,m,*rGY,F"))]
+        (match_operand:V64 1 "input_operand"    "GY,e,W#F,e,*rGY,m,*rGY,DF"))]
   "TARGET_FPU
    && TARGET_ARCH64
    && (register_operand (operands[0], <V64:MODE>mode)
@@ -2643,22 +2643,17 @@
   stx\t%r1, %0"
   [(set_attr "type" "*,load,store")])
 
-;; This pattern build DFmode constants in integer registers.
+;; This pattern builds V64mode constants in integer registers.
 (define_split
-  [(set (match_operand:DF 0 "register_operand" "")
-        (match_operand:DF 1 "const_double_operand" ""))]
+  [(set (match_operand:V64 0 "register_operand" "")
+        (match_operand:V64 1 "const_double_or_vector_operand" ""))]
   "TARGET_FPU
    && (GET_CODE (operands[0]) == REG
        && REGNO (operands[0]) < 32)
-   && ! const_zero_operand(operands[1], DFmode)
+   && ! const_zero_operand (operands[1], GET_MODE (operands[0]))
    && reload_completed"
   [(clobber (const_int 0))]
 {
-  REAL_VALUE_TYPE r;
-  long l[2];
-
-  REAL_VALUE_FROM_CONST_DOUBLE (r, operands[1]);
-  REAL_VALUE_TO_TARGET_DOUBLE (r, l);
   operands[0] = gen_rtx_raw_REG (DImode, REGNO (operands[0]));
 
   if (TARGET_ARCH64)
@@ -2666,31 +2661,34 @@
 #if HOST_BITS_PER_WIDE_INT == 32
       gcc_unreachable ();
 #else
-      HOST_WIDE_INT val;
-
-      val = ((HOST_WIDE_INT)(unsigned long)l[1] |
-             ((HOST_WIDE_INT)(unsigned long)l[0] << 32));
-      emit_insn (gen_movdi (operands[0], gen_int_mode (val, DImode)));
+      enum machine_mode mode = GET_MODE (operands[1]);
+      rtx tem = simplify_subreg (DImode, operands[1], mode, 0);
+      emit_insn (gen_movdi (operands[0], tem));
 #endif
     }
   else
     {
-      emit_insn (gen_movsi (gen_highpart (SImode, operands[0]),
-                           gen_int_mode (l[0], SImode)));
+      enum machine_mode mode = GET_MODE (operands[1]);
+      rtx hi = simplify_subreg (SImode, operands[1], mode, 0);
+      rtx lo = simplify_subreg (SImode, operands[1], mode, 4);
+
+      gcc_assert (GET_CODE (hi) == CONST_INT);
+      gcc_assert (GET_CODE (lo) == CONST_INT);
+
+      emit_insn (gen_movsi (gen_highpart (SImode, operands[0]), hi));
 
       /* Slick... but this trick loses if this subreg constant part
          can be done in one insn.  */
-      if (l[1] == l[0]
-         && ! SPARC_SETHI32_P (l[0])
-         && ! SPARC_SIMM13_P (l[0]))
+      if (lo == hi
+         && ! SPARC_SETHI32_P (INTVAL (hi))
+         && ! SPARC_SIMM13_P (INTVAL (hi)))
         {
           emit_insn (gen_movsi (gen_lowpart (SImode, operands[0]),
                                gen_highpart (SImode, operands[0])));
         }
       else
         {
-          emit_insn (gen_movsi (gen_lowpart (SImode, operands[0]),
-                               gen_int_mode (l[1], SImode)));
+          emit_insn (gen_movsi (gen_lowpart (SImode, operands[0]), lo));
         }
     }
   DONE;

diff --git a/gcc/doc/md.texi b/gcc/doc/md.texi
index 528c6a1c34f6db7ccc2c70bb7a6b7f530c253236..3505dea45af6734f72e204a96e95e1bafd9363b7 100644 (file)
--- a/gcc/doc/md.texi
+++ b/gcc/doc/md.texi
@@ -2738,6 +2738,9 @@ when the Visual Instruction Set is available.
 @item h
 64-bit global or out register for the SPARC-V8+ architecture.
 
+@item D
+A vector constant
+
 @item I
 Signed 13-bit constant