Message ID | 1480939153-3785-3-git-send-email-claziss@synopsys.com |
---|---|
State | New |
Headers | show |
* 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 >
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 --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)] ""