diff mbox

[2/2,ARC] Remove old prof patterns.

Message ID 1480939153-3785-3-git-send-email-claziss@synopsys.com
State New
Headers show

Commit Message

Claudiu Zissulescu Dec. 5, 2016, 11:59 a.m. UTC 
Cleanup old patterns.

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

	* config/arc/arc.md (call_prof): Remove.
	(call_value_prof): Likewise.
	(sibcall_prof): Likewise.
	(sibcall_value_prof): Likewise.
---
 gcc/config/arc/arc.md | 63 ---------------------------------------------------
 1 file changed, 63 deletions(-)

Comments

Andrew Burgess Dec. 14, 2016, 12:14 p.m. UTC | #1 
* Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com> [2016-12-05 12:59:13 +0100]:

> Cleanup old patterns.
> 
> gcc/
> 2016-10-10  Claudiu Zissulescu  <claziss@synopsys.com>
> 
> 	* config/arc/arc.md (call_prof): Remove.
> 	(call_value_prof): Likewise.
> 	(sibcall_prof): Likewise.
> 	(sibcall_value_prof): Likewise.

Looks fine.

Thanks,
Andrew




> ---
>  gcc/config/arc/arc.md | 63 ---------------------------------------------------
>  1 file changed, 63 deletions(-)
> 
> diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
> index 8e26238..0648ff3 100644
> --- a/gcc/config/arc/arc.md
> +++ b/gcc/config/arc/arc.md
> @@ -4140,20 +4140,6 @@
>     (set_attr "predicable" "no,no,yes,yes,no,yes,no,yes")
>     (set_attr "length" "*,*,4,4,4,4,4,8")])
>  
> -(define_insn "call_prof"
> -  [(call (mem:SI (match_operand:SI 0 "symbolic_operand" "Cbr,Cal"))
> -	 (match_operand 1 "" ""))
> -   (clobber (reg:SI 31))
> -   (use (reg:SI 8))
> -   (use (reg:SI 9))]
> -   ""
> -  "@
> -   bl%!%* %P0;2
> -   jl%! %^%S0"
> -  [(set_attr "type" "call,call_no_delay_slot")
> -   (set_attr "predicable" "yes,yes")
> -   (set_attr "length" "4,8")])
> -
>  (define_expand "call_value"
>    ;; operand 2 is stack_size_rtx
>    ;; operand 3 is next_arg_register
> @@ -4203,22 +4189,6 @@
>  ; use it for lack of inter-procedural branch shortening.
>  ; Link-time relaxation would help...
>  
> -
> -(define_insn "call_value_prof"
> -  [(set (match_operand 0 "dest_reg_operand" "=r,r")
> -	(call (mem:SI (match_operand:SI 1 "symbolic_operand" "Cbr,Cal"))
> -	      (match_operand 2 "" "")))
> -   (clobber (reg:SI 31))
> -   (use (reg:SI 8))
> -   (use (reg:SI 9))]
> -   ""
> -  "@
> -   bl%!%* %P1;1
> -   jl%! %^%S1"
> -  [(set_attr "type" "call,call_no_delay_slot")
> -   (set_attr "predicable" "yes,yes")
> -   (set_attr "length" "4,8")])
> -
>  (define_insn "nop"
>    [(const_int 0)]
>    ""
> @@ -4713,39 +4683,6 @@
>     (set_attr "is_SIBCALL" "yes")]
>  )
>  
> -(define_insn "sibcall_prof"
> - [(call (mem:SI (match_operand:SI 0 "call_address_operand" "Cbr,Cal"))
> -	(match_operand 1 "" ""))
> -  (simple_return)
> -  (use (match_operand 2 "" ""))
> -  (use (reg:SI 8))
> -  (use (reg:SI 9))]
> -  ""
> -  "@
> -   b%!%* %P0;2
> -   j%! %^%S0;2"
> -  [(set_attr "type" "call,call_no_delay_slot")
> -   (set_attr "predicable" "yes")
> -   (set_attr "is_SIBCALL" "yes")]
> -)
> -
> -(define_insn "sibcall_value_prof"
> - [(set (match_operand 0 "dest_reg_operand" "")
> -       (call (mem:SI (match_operand:SI 1 "call_address_operand" "Cbr,Cal"))
> -	     (match_operand 2 "" "")))
> -  (simple_return)
> -  (use (match_operand 3 "" ""))
> -  (use (reg:SI 8))
> -  (use (reg:SI 9))]
> -  ""
> -  "@
> -   b%!%* %P1;1
> -   j%! %^%S1;1"
> -  [(set_attr "type" "call,call_no_delay_slot")
> -   (set_attr "predicable" "yes")
> -   (set_attr "is_SIBCALL" "yes")]
> -)
> -
>  (define_expand "prologue"
>    [(pc)]
>    ""
> -- 
> 1.9.1
>
Claudiu Zissulescu Dec. 16, 2016, 1:16 p.m. UTC | #2 
Committed. Thank you for ur review,
Claudiu

> -----Original Message-----
> From: Andrew Burgess [mailto:andrew.burgess@embecosm.com]
> Sent: Wednesday, December 14, 2016 1:14 PM
> To: Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com>
> Cc: gcc-patches@gcc.gnu.org; Francois.Bedard@synopsys.com
> Subject: Re: [PATCH 2/2] [ARC] Remove old prof patterns.
> 
> * Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com> [2016-12-05
> 12:59:13 +0100]:
> 
> > Cleanup old patterns.
> >
> > gcc/
> > 2016-10-10  Claudiu Zissulescu  <claziss@synopsys.com>
> >
> > 	* config/arc/arc.md (call_prof): Remove.
> > 	(call_value_prof): Likewise.
> > 	(sibcall_prof): Likewise.
> > 	(sibcall_value_prof): Likewise.
> 
> Looks fine.
> 
> Thanks,
> Andrew
> 
> 
> 
> 
> > ---
> >  gcc/config/arc/arc.md | 63 ---------------------------------------------------
> >  1 file changed, 63 deletions(-)
> >
> > diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
> > index 8e26238..0648ff3 100644
> > --- a/gcc/config/arc/arc.md
> > +++ b/gcc/config/arc/arc.md
> > @@ -4140,20 +4140,6 @@
> >     (set_attr "predicable" "no,no,yes,yes,no,yes,no,yes")
> >     (set_attr "length" "*,*,4,4,4,4,4,8")])
> >
> > -(define_insn "call_prof"
> > -  [(call (mem:SI (match_operand:SI 0 "symbolic_operand" "Cbr,Cal"))
> > -	 (match_operand 1 "" ""))
> > -   (clobber (reg:SI 31))
> > -   (use (reg:SI 8))
> > -   (use (reg:SI 9))]
> > -   ""
> > -  "@
> > -   bl%!%* %P0;2
> > -   jl%! %^%S0"
> > -  [(set_attr "type" "call,call_no_delay_slot")
> > -   (set_attr "predicable" "yes,yes")
> > -   (set_attr "length" "4,8")])
> > -
> >  (define_expand "call_value"
> >    ;; operand 2 is stack_size_rtx
> >    ;; operand 3 is next_arg_register
> > @@ -4203,22 +4189,6 @@
> >  ; use it for lack of inter-procedural branch shortening.
> >  ; Link-time relaxation would help...
> >
> > -
> > -(define_insn "call_value_prof"
> > -  [(set (match_operand 0 "dest_reg_operand" "=r,r")
> > -	(call (mem:SI (match_operand:SI 1 "symbolic_operand" "Cbr,Cal"))
> > -	      (match_operand 2 "" "")))
> > -   (clobber (reg:SI 31))
> > -   (use (reg:SI 8))
> > -   (use (reg:SI 9))]
> > -   ""
> > -  "@
> > -   bl%!%* %P1;1
> > -   jl%! %^%S1"
> > -  [(set_attr "type" "call,call_no_delay_slot")
> > -   (set_attr "predicable" "yes,yes")
> > -   (set_attr "length" "4,8")])
> > -
> >  (define_insn "nop"
> >    [(const_int 0)]
> >    ""
> > @@ -4713,39 +4683,6 @@
> >     (set_attr "is_SIBCALL" "yes")]
> >  )
> >
> > -(define_insn "sibcall_prof"
> > - [(call (mem:SI (match_operand:SI 0 "call_address_operand" "Cbr,Cal"))
> > -	(match_operand 1 "" ""))
> > -  (simple_return)
> > -  (use (match_operand 2 "" ""))
> > -  (use (reg:SI 8))
> > -  (use (reg:SI 9))]
> > -  ""
> > -  "@
> > -   b%!%* %P0;2
> > -   j%! %^%S0;2"
> > -  [(set_attr "type" "call,call_no_delay_slot")
> > -   (set_attr "predicable" "yes")
> > -   (set_attr "is_SIBCALL" "yes")]
> > -)
> > -
> > -(define_insn "sibcall_value_prof"
> > - [(set (match_operand 0 "dest_reg_operand" "")
> > -       (call (mem:SI (match_operand:SI 1 "call_address_operand" "Cbr,Cal"))
> > -	     (match_operand 2 "" "")))
> > -  (simple_return)
> > -  (use (match_operand 3 "" ""))
> > -  (use (reg:SI 8))
> > -  (use (reg:SI 9))]
> > -  ""
> > -  "@
> > -   b%!%* %P1;1
> > -   j%! %^%S1;1"
> > -  [(set_attr "type" "call,call_no_delay_slot")
> > -   (set_attr "predicable" "yes")
> > -   (set_attr "is_SIBCALL" "yes")]
> > -)
> > -
> >  (define_expand "prologue"
> >    [(pc)]
> >    ""
> > --
> > 1.9.1
> >
diff mbox

Patch

diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index 8e26238..0648ff3 100644
--- a/gcc/config/arc/arc.md
+++ b/gcc/config/arc/arc.md
@@ -4140,20 +4140,6 @@ 
    (set_attr "predicable" "no,no,yes,yes,no,yes,no,yes")
    (set_attr "length" "*,*,4,4,4,4,4,8")])
 
-(define_insn "call_prof"
-  [(call (mem:SI (match_operand:SI 0 "symbolic_operand" "Cbr,Cal"))
-	 (match_operand 1 "" ""))
-   (clobber (reg:SI 31))
-   (use (reg:SI 8))
-   (use (reg:SI 9))]
-   ""
-  "@
-   bl%!%* %P0;2
-   jl%! %^%S0"
-  [(set_attr "type" "call,call_no_delay_slot")
-   (set_attr "predicable" "yes,yes")
-   (set_attr "length" "4,8")])
-
 (define_expand "call_value"
   ;; operand 2 is stack_size_rtx
   ;; operand 3 is next_arg_register
@@ -4203,22 +4189,6 @@ 
 ; use it for lack of inter-procedural branch shortening.
 ; Link-time relaxation would help...
 
-
-(define_insn "call_value_prof"
-  [(set (match_operand 0 "dest_reg_operand" "=r,r")
-	(call (mem:SI (match_operand:SI 1 "symbolic_operand" "Cbr,Cal"))
-	      (match_operand 2 "" "")))
-   (clobber (reg:SI 31))
-   (use (reg:SI 8))
-   (use (reg:SI 9))]
-   ""
-  "@
-   bl%!%* %P1;1
-   jl%! %^%S1"
-  [(set_attr "type" "call,call_no_delay_slot")
-   (set_attr "predicable" "yes,yes")
-   (set_attr "length" "4,8")])
-
 (define_insn "nop"
   [(const_int 0)]
   ""
@@ -4713,39 +4683,6 @@ 
    (set_attr "is_SIBCALL" "yes")]
 )
 
-(define_insn "sibcall_prof"
- [(call (mem:SI (match_operand:SI 0 "call_address_operand" "Cbr,Cal"))
-	(match_operand 1 "" ""))
-  (simple_return)
-  (use (match_operand 2 "" ""))
-  (use (reg:SI 8))
-  (use (reg:SI 9))]
-  ""
-  "@
-   b%!%* %P0;2
-   j%! %^%S0;2"
-  [(set_attr "type" "call,call_no_delay_slot")
-   (set_attr "predicable" "yes")
-   (set_attr "is_SIBCALL" "yes")]
-)
-
-(define_insn "sibcall_value_prof"
- [(set (match_operand 0 "dest_reg_operand" "")
-       (call (mem:SI (match_operand:SI 1 "call_address_operand" "Cbr,Cal"))
-	     (match_operand 2 "" "")))
-  (simple_return)
-  (use (match_operand 3 "" ""))
-  (use (reg:SI 8))
-  (use (reg:SI 9))]
-  ""
-  "@
-   b%!%* %P1;1
-   j%! %^%S1;1"
-  [(set_attr "type" "call,call_no_delay_slot")
-   (set_attr "predicable" "yes")
-   (set_attr "is_SIBCALL" "yes")]
-)
-
 (define_expand "prologue"
   [(pc)]
   ""