[09/43] i386: Emulate MMX <any_logic><mode>3 with SSE

Message ID 20190209132352.1828-10-hjl.tools@gmail.com
State Superseded
Headers show
Series
  • V2: Emulate MMX intrinsics with SSE
Related show

Commit Message

H.J. Lu Feb. 9, 2019, 1:23 p.m.
Emulate MMX <any_logic><mode>3 with SSE.  Only SSE register source
operand is allowed.

	PR target/89021
	* config/i386/mmx.md (any_logic:<code><mode>3): New.
	(any_logic:*mmx_<code><mode>3): Also allow TARGET_MMX_WITH_SSE.
	Add SSE support.
---
 gcc/config/i386/mmx.md | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

-- 
2.20.1

Comments

Uros Bizjak Feb. 10, 2019, 10:27 a.m. | #1
On 2/9/19, H.J. Lu <hjl.tools@gmail.com> wrote:
> Emulate MMX <any_logic><mode>3 with SSE.  Only SSE register source

> operand is allowed.

>

> 	PR target/89021

> 	* config/i386/mmx.md (any_logic:<code><mode>3): New.

> 	(any_logic:*mmx_<code><mode>3): Also allow TARGET_MMX_WITH_SSE.

> 	Add SSE support.


OK.

Uros.

> ---

>  gcc/config/i386/mmx.md | 27 ++++++++++++++++++++-------

>  1 file changed, 20 insertions(+), 7 deletions(-)

>

> diff --git a/gcc/config/i386/mmx.md b/gcc/config/i386/mmx.md

> index 69c66e968b5..fae2e43af24 100644

> --- a/gcc/config/i386/mmx.md

> +++ b/gcc/config/i386/mmx.md

> @@ -1110,15 +1110,28 @@

>    "TARGET_MMX"

>    "ix86_fixup_binary_operands_no_copy (<CODE>, <MODE>mode, operands);")

>

> +(define_expand "<code><mode>3"

> +  [(set (match_operand:MMXMODEI 0 "register_operand")

> +	(any_logic:MMXMODEI

> +	  (match_operand:MMXMODEI 1 "nonimmediate_operand")

> +	  (match_operand:MMXMODEI 2 "nonimmediate_operand")))]

> +  "TARGET_MMX_WITH_SSE"

> +  "ix86_fixup_binary_operands_no_copy (<CODE>, <MODE>mode, operands);")

> +

>  (define_insn "*mmx_<code><mode>3"

> -  [(set (match_operand:MMXMODEI 0 "register_operand" "=y")

> +  [(set (match_operand:MMXMODEI 0 "register_operand" "=y,Yx,Yy")

>          (any_logic:MMXMODEI

> -	  (match_operand:MMXMODEI 1 "nonimmediate_operand" "%0")

> -	  (match_operand:MMXMODEI 2 "nonimmediate_operand" "ym")))]

> -  "TARGET_MMX && ix86_binary_operator_ok (<CODE>, <MODE>mode, operands)"

> -  "p<logic>\t{%2, %0|%0, %2}"

> -  [(set_attr "type" "mmxadd")

> -   (set_attr "mode" "DI")])

> +	  (match_operand:MMXMODEI 1 "nonimmediate_operand" "%0,0,Yy")

> +	  (match_operand:MMXMODEI 2 "nonimmediate_operand" "ym,Yx,Yy")))]

> +  "(TARGET_MMX || TARGET_MMX_WITH_SSE)

> +   && ix86_binary_operator_ok (<CODE>, <MODE>mode, operands)"

> +  "@

> +   p<logic>\t{%2, %0|%0, %2}

> +   p<logic>\t{%2, %0|%0, %2}

> +   vp<logic>\t{%2, %1, %0|%0, %1, %2}"

> +  [(set_attr "mmx_isa" "native,x64_noavx,x64_avx")

> +   (set_attr "type" "mmxadd,sselog,sselog")

> +   (set_attr "mode" "DI,TI,TI")])

>

>  ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;

>  ;;

> --

> 2.20.1

>

>

Patch

diff --git a/gcc/config/i386/mmx.md b/gcc/config/i386/mmx.md
index 69c66e968b5..fae2e43af24 100644
--- a/gcc/config/i386/mmx.md
+++ b/gcc/config/i386/mmx.md
@@ -1110,15 +1110,28 @@ 
   "TARGET_MMX"
   "ix86_fixup_binary_operands_no_copy (<CODE>, <MODE>mode, operands);")
 
+(define_expand "<code><mode>3"
+  [(set (match_operand:MMXMODEI 0 "register_operand")
+	(any_logic:MMXMODEI
+	  (match_operand:MMXMODEI 1 "nonimmediate_operand")
+	  (match_operand:MMXMODEI 2 "nonimmediate_operand")))]
+  "TARGET_MMX_WITH_SSE"
+  "ix86_fixup_binary_operands_no_copy (<CODE>, <MODE>mode, operands);")
+
 (define_insn "*mmx_<code><mode>3"
-  [(set (match_operand:MMXMODEI 0 "register_operand" "=y")
+  [(set (match_operand:MMXMODEI 0 "register_operand" "=y,Yx,Yy")
         (any_logic:MMXMODEI
-	  (match_operand:MMXMODEI 1 "nonimmediate_operand" "%0")
-	  (match_operand:MMXMODEI 2 "nonimmediate_operand" "ym")))]
-  "TARGET_MMX && ix86_binary_operator_ok (<CODE>, <MODE>mode, operands)"
-  "p<logic>\t{%2, %0|%0, %2}"
-  [(set_attr "type" "mmxadd")
-   (set_attr "mode" "DI")])
+	  (match_operand:MMXMODEI 1 "nonimmediate_operand" "%0,0,Yy")
+	  (match_operand:MMXMODEI 2 "nonimmediate_operand" "ym,Yx,Yy")))]
+  "(TARGET_MMX || TARGET_MMX_WITH_SSE)
+   && ix86_binary_operator_ok (<CODE>, <MODE>mode, operands)"
+  "@
+   p<logic>\t{%2, %0|%0, %2}
+   p<logic>\t{%2, %0|%0, %2}
+   vp<logic>\t{%2, %1, %0|%0, %1, %2}"
+  [(set_attr "mmx_isa" "native,x64_noavx,x64_avx")
+   (set_attr "type" "mmxadd,sselog,sselog")
+   (set_attr "mode" "DI,TI,TI")])
 
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;;