[06/11] i386: Update unexpected empty split condition

Message ID 513fefb141a66df5675c1cb0210f7f88ace4c740.1622179420.git.linkw@linux.ibm.com
State New
Headers show
Series
  • Fix up some unexpected empty split conditions
Related show

Commit Message

Jeff Law via Gcc-patches June 2, 2021, 5:04 a.m.
gcc/ChangeLog:

	* config/i386/i386.md (*load_tp_x32_zext, *add_tp_x32_zext,
	*tls_dynamic_gnu2_combine_32): Fix empty split condition.
	* config/i386/sse.md (*<sse2_avx2>_pmovmskb_lt,
	*<sse2_avx2>_pmovmskb_zext_lt, *sse2_pmovmskb_ext_lt,
	*<sse4_1_avx2>_pblendvb_lt): Likewise.
---
 gcc/config/i386/i386.md | 6 +++---
 gcc/config/i386/sse.md  | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

-- 
2.17.1

Comments

Jeff Law via Gcc-patches June 2, 2021, 6:28 a.m. | #1
On Wed, Jun 2, 2021 at 7:05 AM Kewen Lin <linkw@linux.ibm.com> wrote:
>

> gcc/ChangeLog:

>

>         * config/i386/i386.md (*load_tp_x32_zext, *add_tp_x32_zext,

>         *tls_dynamic_gnu2_combine_32): Fix empty split condition.

>         * config/i386/sse.md (*<sse2_avx2>_pmovmskb_lt,

>         *<sse2_avx2>_pmovmskb_zext_lt, *sse2_pmovmskb_ext_lt,

>         *<sse4_1_avx2>_pblendvb_lt): Likewise.


OK.

Thanks,
Uros.

> ---

>  gcc/config/i386/i386.md | 6 +++---

>  gcc/config/i386/sse.md  | 8 ++++----

>  2 files changed, 7 insertions(+), 7 deletions(-)

>

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

> index 9ff35d9a607..545d048906d 100644

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

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

> @@ -15712,7 +15712,7 @@ (define_insn_and_split "*load_tp_x32_zext"

>           (unspec:SI [(const_int 0)] UNSPEC_TP)))]

>    "TARGET_X32"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0)

>         (zero_extend:DI (match_dup 1)))]

>  {

> @@ -15750,7 +15750,7 @@ (define_insn_and_split "*add_tp_x32_zext"

>     (clobber (reg:CC FLAGS_REG))]

>    "TARGET_X32"

>    "#"

> -  ""

> +  "&& 1"

>    [(parallel

>       [(set (match_dup 0)

>            (zero_extend:DI

> @@ -15841,7 +15841,7 @@ (define_insn_and_split "*tls_dynamic_gnu2_combine_32"

>     (clobber (reg:CC FLAGS_REG))]

>    "!TARGET_64BIT && TARGET_GNU2_TLS"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0) (match_dup 5))]

>  {

>    operands[5] = can_create_pseudo_p () ? gen_reg_rtx (Pmode) : operands[0];

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

> index 9d3728d1cb0..a9d78030119 100644

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

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

> @@ -16467,7 +16467,7 @@ (define_insn_and_split "*<sse2_avx2>_pmovmskb_lt"

>           UNSPEC_MOVMSK))]

>    "TARGET_SSE2"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0)

>         (unspec:SI [(match_dup 1)] UNSPEC_MOVMSK))]

>    ""

> @@ -16489,7 +16489,7 @@ (define_insn_and_split "*<sse2_avx2>_pmovmskb_zext_lt"

>             UNSPEC_MOVMSK)))]

>    "TARGET_64BIT && TARGET_SSE2"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0)

>         (zero_extend:DI (unspec:SI [(match_dup 1)] UNSPEC_MOVMSK)))]

>    ""

> @@ -16511,7 +16511,7 @@ (define_insn_and_split "*sse2_pmovmskb_ext_lt"

>             UNSPEC_MOVMSK)))]

>    "TARGET_64BIT && TARGET_SSE2"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0)

>         (sign_extend:DI (unspec:SI [(match_dup 1)] UNSPEC_MOVMSK)))]

>    ""

> @@ -17769,7 +17769,7 @@ (define_insn_and_split "*<sse4_1_avx2>_pblendvb_lt"

>           UNSPEC_BLENDV))]

>    "TARGET_SSE4_1"

>    "#"

> -  ""

> +  "&& 1"

>    [(set (match_dup 0)

>         (unspec:VI1_AVX2

>          [(match_dup 1) (match_dup 2) (match_dup 3)] UNSPEC_BLENDV))]

> --

> 2.17.1

>

Patch

diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
index 9ff35d9a607..545d048906d 100644
--- a/gcc/config/i386/i386.md
+++ b/gcc/config/i386/i386.md
@@ -15712,7 +15712,7 @@  (define_insn_and_split "*load_tp_x32_zext"
 	  (unspec:SI [(const_int 0)] UNSPEC_TP)))]
   "TARGET_X32"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0)
 	(zero_extend:DI (match_dup 1)))]
 {
@@ -15750,7 +15750,7 @@  (define_insn_and_split "*add_tp_x32_zext"
    (clobber (reg:CC FLAGS_REG))]
   "TARGET_X32"
   "#"
-  ""
+  "&& 1"
   [(parallel
      [(set (match_dup 0)
      	   (zero_extend:DI
@@ -15841,7 +15841,7 @@  (define_insn_and_split "*tls_dynamic_gnu2_combine_32"
    (clobber (reg:CC FLAGS_REG))]
   "!TARGET_64BIT && TARGET_GNU2_TLS"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0) (match_dup 5))]
 {
   operands[5] = can_create_pseudo_p () ? gen_reg_rtx (Pmode) : operands[0];
diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md
index 9d3728d1cb0..a9d78030119 100644
--- a/gcc/config/i386/sse.md
+++ b/gcc/config/i386/sse.md
@@ -16467,7 +16467,7 @@  (define_insn_and_split "*<sse2_avx2>_pmovmskb_lt"
 	  UNSPEC_MOVMSK))]
   "TARGET_SSE2"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0)
 	(unspec:SI [(match_dup 1)] UNSPEC_MOVMSK))]
   ""
@@ -16489,7 +16489,7 @@  (define_insn_and_split "*<sse2_avx2>_pmovmskb_zext_lt"
 	    UNSPEC_MOVMSK)))]
   "TARGET_64BIT && TARGET_SSE2"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0)
 	(zero_extend:DI (unspec:SI [(match_dup 1)] UNSPEC_MOVMSK)))]
   ""
@@ -16511,7 +16511,7 @@  (define_insn_and_split "*sse2_pmovmskb_ext_lt"
 	    UNSPEC_MOVMSK)))]
   "TARGET_64BIT && TARGET_SSE2"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0)
 	(sign_extend:DI (unspec:SI [(match_dup 1)] UNSPEC_MOVMSK)))]
   ""
@@ -17769,7 +17769,7 @@  (define_insn_and_split "*<sse4_1_avx2>_pblendvb_lt"
 	  UNSPEC_BLENDV))]
   "TARGET_SSE4_1"
   "#"
-  ""
+  "&& 1"
   [(set (match_dup 0)
 	(unspec:VI1_AVX2
 	 [(match_dup 1) (match_dup 2) (match_dup 3)] UNSPEC_BLENDV))]