Hello!

2013-03-25  Uros Bizjak  <ubiz...@gmail.com>

        * config/i386/i386.md (*zero_extendsidi2): Merge with
        *zero_extendsidi2_rex64.  Use x64 and nox64 isa attributes.
        * config/i386/predicates.md (x86_64_zext_operand): Rename from
        x86_64_zext_general_operand.  Use nonimmediate_operand on 32bit
        targets.  Clarify comment.

Tested on x86_64-pc-linux-gnu {,-m32} and committed to mainline SVN.

Uros.
Index: i386.md
===================================================================
--- i386.md     (revision 197053)
+++ i386.md     (working copy)
@@ -3135,13 +3135,13 @@
   [(set (match_operand:DI 0 "nonimmediate_operand")
        (zero_extend:DI (match_operand:SI 1 "nonimmediate_operand")))])
 
-(define_insn "*zero_extendsidi2_rex64"
+(define_insn "*zero_extendsidi2"
   [(set (match_operand:DI 0 "nonimmediate_operand"
-                       "=r  ,o,?*Ym,?*y,?*Yi,?*x")
+                       "=r,?r,?o,r   ,o,?*Ym,?*y,?*Yi,?*x")
        (zero_extend:DI
-        (match_operand:SI 1 "x86_64_zext_general_operand"
-                       "rmWz,0,r   ,m  ,r   ,m")))]
-  "TARGET_64BIT"
+        (match_operand:SI 1 "x86_64_zext_operand"
+                       "0 ,rm,r ,rmWz,0,r   ,m  ,r   ,m")))]
+  ""
 {
   switch (get_attr_type (insn))
     {
@@ -3164,30 +3164,40 @@
       gcc_unreachable ();
     }
 }
-  [(set_attr "type" "imovx,multi,mmxmov,mmxmov,ssemov,ssemov")
-   (set_attr "prefix" "orig,*,orig,orig,maybe_vex,maybe_vex")
-   (set_attr "prefix_0f" "0,*,*,*,*,*")
-   (set_attr "mode" "SI,SI,DI,DI,TI,TI")])
+  [(set (attr "isa")
+     (cond [(eq_attr "alternative" "0,1,2")
+             (const_string "nox64")
+           (eq_attr "alternative" "3")
+             (const_string "x64")
+           (eq_attr "alternative" "8")
+             (const_string "sse2")
+          ]
+          (const_string "*")))
+   (set (attr "type")
+     (cond [(eq_attr "alternative" "0,1,2,4")
+             (const_string "multi")
+           (eq_attr "alternative" "5,6")
+             (const_string "mmxmov")
+           (eq_attr "alternative" "7,8")
+             (const_string "ssemov")
+          ]
+          (const_string "imovx")))
+   (set (attr "prefix")
+     (if_then_else (eq_attr "type" "ssemov")
+       (const_string "maybe_vex")
+       (const_string "orig")))
+   (set (attr "prefix_0f")
+     (if_then_else (eq_attr "type" "imovx")
+       (const_string "0")
+       (const_string "*")))
+   (set (attr "mode")
+     (cond [(eq_attr "alternative" "5,6")
+             (const_string "DI")
+           (eq_attr "alternative" "7,8")
+             (const_string "TI")
+          ]
+          (const_string "SI")))])
 
-(define_insn "*zero_extendsidi2"
-  [(set (match_operand:DI 0 "nonimmediate_operand"
-                       "=ro,?r,?o,?*Ym,?*y,?*Yi,?*x")
-       (zero_extend:DI (match_operand:SI 1 "nonimmediate_operand"
-                       "0  ,rm,r ,r   ,m  ,r   ,m")))]
-  "!TARGET_64BIT"
-  "@
-   #
-   #
-   #
-   movd\t{%1, %0|%0, %1}
-   movd\t{%1, %0|%0, %1}
-   %vmovd\t{%1, %0|%0, %1}
-   %vmovd\t{%1, %0|%0, %1}"
-  [(set_attr "isa" "*,*,*,*,*,*,sse2")
-   (set_attr "type" "multi,multi,multi,mmxmov,mmxmov,ssemov,ssemov")
-   (set_attr "prefix" "*,*,*,orig,orig,maybe_vex,maybe_vex")
-   (set_attr "mode" "SI,SI,SI,DI,DI,TI,TI")])
-
 (define_split
   [(set (match_operand:DI 0 "memory_operand")
        (zero_extend:DI (match_operand:SI 1 "memory_operand")))]
Index: predicates.md
===================================================================
--- predicates.md       (revision 197053)
+++ predicates.md       (working copy)
@@ -311,15 +311,15 @@
         (match_operand 0 "x86_64_immediate_operand"))
     (match_operand 0 "general_operand")))
 
-;; Return true if OP is general operand representable on x86_64
-;; as zero extended constant.  This predicate is used in zero-extending
-;; conversion operations that require non-VOIDmode immediate operands.
-(define_predicate "x86_64_zext_general_operand"
+;; Return true if OP is representable on x86_64 as zero-extended operand.
+;; This predicate is used in zero-extending conversion operations that
+;; require non-VOIDmode immediate operands.
+(define_predicate "x86_64_zext_operand"
   (if_then_else (match_test "TARGET_64BIT")
     (ior (match_operand 0 "nonimmediate_operand")
         (and (match_operand 0 "x86_64_zext_immediate_operand")
              (match_test "GET_MODE (op) != VOIDmode")))
-    (match_operand 0 "general_operand")))
+    (match_operand 0 "nonimmediate_operand")))
 
 ;; Return true if OP is general operand representable on x86_64
 ;; as either sign extended or zero extended constant.

Reply via email to