diff mbox series

bpf: remove useless define_insn for extendsisi2

Message ID 20230815184908.7474-1-david.faust@oracle.com
State Unresolved
Headers
Series bpf: remove useless define_insn for extendsisi2 |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

David Faust Aug. 15, 2023, 6:49 p.m. UTC 
  This define_insn is never used, since a sign-extend to the same mode is
just a move, so delete it.

Tested on x86_64-linux-gnu host for bpf-unknown-none target.

gcc/

	* config/bpf/bpf.md (extendsisi2): Delete useless define_insn.
---
 gcc/config/bpf/bpf.md | 7 -------
 1 file changed, 7 deletions(-)

Comments

Jose E. Marchesi Aug. 15, 2023, 7:03 p.m. UTC | #1 
OK.
Thanks!

> This define_insn is never used, since a sign-extend to the same mode is
> just a move, so delete it.
>
> Tested on x86_64-linux-gnu host for bpf-unknown-none target.
>
> gcc/
>
> 	* config/bpf/bpf.md (extendsisi2): Delete useless define_insn.
> ---
>  gcc/config/bpf/bpf.md | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/gcc/config/bpf/bpf.md b/gcc/config/bpf/bpf.md
> index e0a42b9f939..a64de1095ed 100644
> --- a/gcc/config/bpf/bpf.md
> +++ b/gcc/config/bpf/bpf.md
> @@ -350,13 +350,6 @@ (define_insn "extendqidi2"
>     {ldxsb\t%0,%1|%0 = *(s8 *) (%1)}"
>    [(set_attr "type" "alu,ldx")])
>  
> -(define_insn "extendsisi2"
> -  [(set (match_operand:SI 0 "register_operand" "=r")
> -        (sign_extend:SI (match_operand:SI 1 "register_operand" "r")))]
> -  "bpf_has_smov"
> -  "{movs32\t%0,%1,32|%w0 = (s32) %w1}"
> -  [(set_attr "type" "alu")])
> -
>  (define_insn "extendhisi2"
>    [(set (match_operand:SI 0 "register_operand" "=r")
>          (sign_extend:SI (match_operand:HI 1 "register_operand" "r")))]
diff mbox series

Patch

diff --git a/gcc/config/bpf/bpf.md b/gcc/config/bpf/bpf.md
index e0a42b9f939..a64de1095ed 100644
--- a/gcc/config/bpf/bpf.md
+++ b/gcc/config/bpf/bpf.md
@@ -350,13 +350,6 @@  (define_insn "extendqidi2"
    {ldxsb\t%0,%1|%0 = *(s8 *) (%1)}"
   [(set_attr "type" "alu,ldx")])
 
-(define_insn "extendsisi2"
-  [(set (match_operand:SI 0 "register_operand" "=r")
-        (sign_extend:SI (match_operand:SI 1 "register_operand" "r")))]
-  "bpf_has_smov"
-  "{movs32\t%0,%1,32|%w0 = (s32) %w1}"
-  [(set_attr "type" "alu")])
-
 (define_insn "extendhisi2"
   [(set (match_operand:SI 0 "register_operand" "=r")
         (sign_extend:SI (match_operand:HI 1 "register_operand" "r")))]