public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] [ARC] Add simple shift/rotate ops.
@ 2016-06-17  9:13 Claudiu Zissulescu
  2016-09-27 14:29 ` Andrew Burgess
  0 siblings, 1 reply; 3+ messages in thread
From: Claudiu Zissulescu @ 2016-06-17  9:13 UTC (permalink / raw)
  To: gcc-patches; +Cc: Claudiu.Zissulescu, gnu, Francois.Bedard

Basic ARC cpus are having only simple shift operations. Here they are.

OK to apply?
Claudiu

gcc/
2016-06-09  Claudiu Zissulescu  <claziss@synopsys.com>

	* config/arc/arc.md (*rotrsi3_cnt1): New pattern,
	(*ashlsi2_cnt1, *lshrsi3_cnt1, *ashrsi3_cnt1): Likewise.
---
 gcc/config/arc/arc.md | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index 852f0e0..a095ba1 100644
--- a/gcc/config/arc/arc.md
+++ b/gcc/config/arc/arc.md
@@ -6219,6 +6219,46 @@
 		   (zero_extract:SI (match_dup 1) (match_dup 5) (match_dup 7)))])
    (match_dup 1)])
 
+(define_insn "*rotrsi3_cnt1"
+  [(set (match_operand:SI 0 "dest_reg_operand"             "=w")
+	(rotatert:SI (match_operand:SI 1 "register_operand" "c")
+		     (const_int 1)))]
+  ""
+  "ror %0,%1%&"
+  [(set_attr "type" "shift")
+   (set_attr "predicable" "no")
+   (set_attr "length" "4")])
+
+(define_insn "*ashlsi2_cnt1"
+  [(set (match_operand:SI 0 "dest_reg_operand"           "=Rcqq,w")
+	(ashift:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
+		   (const_int 1)))]
+  ""
+  "asl%? %0,%1%&"
+  [(set_attr "type" "shift")
+   (set_attr "iscompact" "maybe,false")
+   (set_attr "predicable" "no,no")])
+
+(define_insn "*lshrsi3_cnt1"
+  [(set (match_operand:SI 0 "dest_reg_operand"             "=Rcqq,w")
+	(lshiftrt:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
+		     (const_int 1)))]
+  ""
+  "lsr%? %0,%1%&"
+  [(set_attr "type" "shift")
+   (set_attr "iscompact" "maybe,false")
+   (set_attr "predicable" "no,no")])
+
+(define_insn "*ashrsi3_cnt1"
+  [(set (match_operand:SI 0 "dest_reg_operand"             "=Rcqq,w")
+	(ashiftrt:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
+		     (const_int 1)))]
+  ""
+  "asr%? %0,%1%&"
+  [(set_attr "type" "shift")
+   (set_attr "iscompact" "maybe,false")
+   (set_attr "predicable" "no,no")])
+
 ;; include the arc-FPX instructions
 (include "fpx.md")
 
-- 
1.9.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] [ARC] Add simple shift/rotate ops.
  2016-06-17  9:13 [PATCH] [ARC] Add simple shift/rotate ops Claudiu Zissulescu
@ 2016-09-27 14:29 ` Andrew Burgess
  2016-09-28 13:54   ` Claudiu Zissulescu
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Burgess @ 2016-09-27 14:29 UTC (permalink / raw)
  To: Claudiu Zissulescu; +Cc: gcc-patches, gnu, Francois.Bedard

* Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com> [2016-06-17 11:13:08 +0200]:

> Basic ARC cpus are having only simple shift operations. Here they are.
> 
> OK to apply?

This looks good to me.

Thanks,
Andrew



> 
> gcc/
> 2016-06-09  Claudiu Zissulescu  <claziss@synopsys.com>
> 
> 	* config/arc/arc.md (*rotrsi3_cnt1): New pattern,
> 	(*ashlsi2_cnt1, *lshrsi3_cnt1, *ashrsi3_cnt1): Likewise.
> ---
>  gcc/config/arc/arc.md | 40 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 40 insertions(+)
> 
> diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
> index 852f0e0..a095ba1 100644
> --- a/gcc/config/arc/arc.md
> +++ b/gcc/config/arc/arc.md
> @@ -6219,6 +6219,46 @@
>  		   (zero_extract:SI (match_dup 1) (match_dup 5) (match_dup 7)))])
>     (match_dup 1)])
>  
> +(define_insn "*rotrsi3_cnt1"
> +  [(set (match_operand:SI 0 "dest_reg_operand"             "=w")
> +	(rotatert:SI (match_operand:SI 1 "register_operand" "c")
> +		     (const_int 1)))]
> +  ""
> +  "ror %0,%1%&"
> +  [(set_attr "type" "shift")
> +   (set_attr "predicable" "no")
> +   (set_attr "length" "4")])
> +
> +(define_insn "*ashlsi2_cnt1"
> +  [(set (match_operand:SI 0 "dest_reg_operand"           "=Rcqq,w")
> +	(ashift:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
> +		   (const_int 1)))]
> +  ""
> +  "asl%? %0,%1%&"
> +  [(set_attr "type" "shift")
> +   (set_attr "iscompact" "maybe,false")
> +   (set_attr "predicable" "no,no")])
> +
> +(define_insn "*lshrsi3_cnt1"
> +  [(set (match_operand:SI 0 "dest_reg_operand"             "=Rcqq,w")
> +	(lshiftrt:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
> +		     (const_int 1)))]
> +  ""
> +  "lsr%? %0,%1%&"
> +  [(set_attr "type" "shift")
> +   (set_attr "iscompact" "maybe,false")
> +   (set_attr "predicable" "no,no")])
> +
> +(define_insn "*ashrsi3_cnt1"
> +  [(set (match_operand:SI 0 "dest_reg_operand"             "=Rcqq,w")
> +	(ashiftrt:SI (match_operand:SI 1 "register_operand" "Rcqq,c")
> +		     (const_int 1)))]
> +  ""
> +  "asr%? %0,%1%&"
> +  [(set_attr "type" "shift")
> +   (set_attr "iscompact" "maybe,false")
> +   (set_attr "predicable" "no,no")])
> +
>  ;; include the arc-FPX instructions
>  (include "fpx.md")
>  
> -- 
> 1.9.1
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] [ARC] Add simple shift/rotate ops.
  2016-09-27 14:29 ` Andrew Burgess
@ 2016-09-28 13:54   ` Claudiu Zissulescu
  0 siblings, 0 replies; 3+ messages in thread
From: Claudiu Zissulescu @ 2016-09-28 13:54 UTC (permalink / raw)
  To: Andrew Burgess; +Cc: gcc-patches, gnu, Francois.Bedard

> 
> This looks good to me.
> 
> Thanks,
> Andrew
> 
Committed r240576.

Thank you for your review,
Claudiu

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-09-28 13:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-17  9:13 [PATCH] [ARC] Add simple shift/rotate ops Claudiu Zissulescu
2016-09-27 14:29 ` Andrew Burgess
2016-09-28 13:54   ` Claudiu Zissulescu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).