annotate gcc/config/riscv/riscv-sr.c @ 145:1830386684a0

gcc-9.2.0
author anatofuz
date Thu, 13 Feb 2020 11:34:05 +0900
parents
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
145
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
1 /* This file is part of GCC.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
2
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
3 GCC is free software; you can redistribute it and/or modify
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
4 it under the terms of the GNU General Public License as published by
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
5 the Free Software Foundation; either version 3, or (at your option)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
6 any later version.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
7
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
8 GCC is distributed in the hope that it will be useful,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
9 but WITHOUT ANY WARRANTY; without even the implied warranty of
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
10 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
11 GNU General Public License for more details.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
12
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
13 You should have received a copy of the GNU General Public License
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
14 along with GCC; see the file COPYING3. If not see
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
15 <http://www.gnu.org/licenses/>. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
16
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
17 /* This file contains code aimed at optimizing function generated with the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
18 use of '-msave-restore. The goal is to identify cases where the call
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
19 out to the save/restore routines are sub-optimal, and remove the calls
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
20 in this case.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
21
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
22 As GCC currently makes the choice between using or not using
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
23 save/restore early on (during the gimple expand pass) once we have
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
24 selected to use save/restore we are stuck with it. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
25
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
26 #define IN_TARGET_CODE 1
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
27
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
28 #include "config.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
29 #include "system.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
30 #include "coretypes.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
31 #include "tm.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
32 #include "rtl.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
33 #include "function.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
34 #include "memmodel.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
35 #include "emit-rtl.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
36 #include "target.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
37 #include "basic-block.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
38 #include "bitmap.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
39 #include "df.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
40 #include "tree.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
41 #include "expr.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
42 #include "cfg.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
43
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
44 /* This file should be included last. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
45 #include "hard-reg-set.h"
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
46
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
47 /* Look in the function prologue for a call to the save stub. Ensure that
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
48 the instruction is as we expect (see detail below) and if the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
49 instruction matches return a pointer to it. Otherwise, return NULL.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
50
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
51 We expect the function prologue to look like this:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
52
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
53 (note NOTE_INSN_BASIC_BLOCK)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
54 (insn (parallel [
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
55 (unspec_volatile [
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
56 (const_int 2 [0x2])
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
57 ] UNSPECV_GPR_SAVE)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
58 (clobber (reg:SI 5 t0))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
59 (clobber (reg:SI 6 t1))])
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
60 (note NOTE_INSN_PROLOGUE_END)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
61
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
62 Between the NOTE_INSN_BASIC_BLOCK and the GPR_SAVE insn we might find
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
63 other notes of type NOTE_INSN_DELETED and/or NOTE_INSN_FUNCTION_BEG.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
64
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
65 The parameter BODY is updated to point to the first instruction after
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
66 the NOTE_INSN_PROLOGUE_END or will be updated to NULL if the prologue
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
67 end note was not found. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
68
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
69 static rtx_insn *
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
70 riscv_sr_match_prologue (rtx_insn **body)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
71 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
72 rtx_insn *insn, *bb_note;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
73 *body = NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
74
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
75 /* Find the prologue end note. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
76 for (insn = get_insns (); insn != NULL; insn = NEXT_INSN (insn))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
77 if (NOTE_P (insn) && NOTE_KIND (insn) == NOTE_INSN_PROLOGUE_END)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
78 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
79 *body = NEXT_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
80 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
81 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
82
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
83 /* If we don't have the prologue end note and at least one instruction
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
84 before it, then this function doesn't have the structure we expect. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
85 if (insn == NULL
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
86 || PREV_INSN (insn) == NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
87 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
88
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
89 /* The INSN is the end of prologue note, before this we expect to find
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
90 one real instruction which makes the prologue, and before that we
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
91 expect to find some number of notes for deleted instructions, the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
92 beginning of the function, and finally a basicblock beginning. The
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
93 following loop checks that this assumption is true. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
94 for (bb_note = PREV_INSN (PREV_INSN (insn));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
95 bb_note != NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
96 bb_note = PREV_INSN (bb_note))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
97 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
98 if (!NOTE_P (bb_note))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
99 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
100 if (NOTE_KIND (bb_note) == NOTE_INSN_BASIC_BLOCK)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
101 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
102 if (NOTE_KIND (bb_note) != NOTE_INSN_DELETED
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
103 && NOTE_KIND (bb_note) != NOTE_INSN_FUNCTION_BEG)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
104 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
105 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
106 if (bb_note == NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
107 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
108
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
109 /* Set INSN to point to the actual interesting prologue instruction. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
110 insn = PREV_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
111 if (INSN_P (insn)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
112 && INSN_CODE (insn) == CODE_FOR_gpr_save
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
113 /* Check this is a call to _riscv_save_0. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
114 && GET_CODE (PATTERN (insn)) == PARALLEL
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
115 && GET_CODE (XVECEXP (PATTERN (insn), 0, 0)) == UNSPEC_VOLATILE
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
116 && (GET_CODE (XVECEXP (XVECEXP (PATTERN (insn), 0, 0), 0, 0))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
117 == CONST_INT)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
118 && INTVAL (XVECEXP (XVECEXP (PATTERN (insn), 0, 0), 0, 0)) == 2)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
119 return insn;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
120
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
121 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
122 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
123
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
124 /* Find the first instruction in the epilogue of the current function, and
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
125 return a pointer to that instruction if, and only if, the epilogue has
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
126 the correct structure that would allow us to optimize out the call to
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
127 _riscv_restore_0. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
128
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
129 static rtx_insn *
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
130 riscv_sr_match_epilogue (void)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
131 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
132 /* Find the first instruction in the epilogue. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
133 rtx_insn *insn, *start;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
134 for (insn = get_insns (); insn != NULL; insn = NEXT_INSN (insn))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
135 if (NOTE_P (insn) && NOTE_KIND (insn) == NOTE_INSN_EPILOGUE_BEG)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
136 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
137 insn = NEXT_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
138 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
139 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
140 if (insn == NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
141 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
142
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
143 /* At this point INSN is the first instruction in the epilogue. A
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
144 standard epilogue (of the form we expect to handle) consists of the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
145 following instructions:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
146
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
147 1. A stack_tiesi or stack_tiedi (for RV32 and RV64 respectively),
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
148
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
149 2. An optional use instruction for the register holding the return
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
150 value. This will be missing in functions with no return value,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
151
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
152 3. A gpr_restore instruction, and
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
153
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
154 4. A jump instruction of type gpr_restore_return. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
155 start = insn;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
156 if (INSN_CODE (insn) != CODE_FOR_stack_tiesi
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
157 && INSN_CODE (insn) != CODE_FOR_stack_tiedi)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
158 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
159
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
160 insn = NEXT_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
161 if (INSN_P (insn) && GET_CODE (PATTERN (insn)) == USE)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
162 insn = NEXT_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
163
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
164 if (!INSN_P (insn) || INSN_CODE (insn) != CODE_FOR_gpr_restore)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
165 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
166
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
167 insn = NEXT_INSN (insn);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
168 if (!INSN_P (insn) || INSN_CODE (insn) != CODE_FOR_gpr_restore_return)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
169 return NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
170
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
171 return start;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
172 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
173
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
174 /* Helper for riscv_remove_unneeded_save_restore_calls. If we match the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
175 prologue instructions but not the epilogue then we might have the case
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
176 where the epilogue has been optimized out due to a call to a no-return
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
177 function. In this case we might be able to remove the prologue too -
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
178 that's what this function does. PROLOGUE is the matched prolgoue
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
179 instruction, by the time this function returns the progloue instruction
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
180 may have been removed. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
181
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
182 static void
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
183 check_for_no_return_call (rtx_insn *prologue)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
184 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
185 /* Check to see if we have the following pattern:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
186
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
187 PROLOGUE instruction
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
188 NOTE_INSN_PROLOGUE_END
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
189 A no-return call instruction
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
190
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
191 If we do, then we can remove the prologue instruction safely. Remember
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
192 that we've already confirmed by this point that the prologue is a call
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
193 to riscv_save_0. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
194
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
195 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
196 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
197 "Prologue matched, checking for no-return epilogue.\n");
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
198
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
199 rtx_insn *tmp = NEXT_INSN (prologue);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
200 if (!NOTE_P (tmp) || NOTE_KIND (tmp) != NOTE_INSN_PROLOGUE_END)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
201 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
202
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
203 /* Skip any extra notes in here, they're most likely just debug. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
204 do
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
205 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
206 tmp = NEXT_INSN (tmp);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
207 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
208 while (tmp != NULL && NOTE_P (tmp));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
209
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
210 if (tmp == NULL || !INSN_P (tmp))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
211 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
212
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
213 bool noreturn_p = find_reg_note (tmp, REG_NORETURN, NULL_RTX) != NULL_RTX;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
214 if (!CALL_P (tmp) || !noreturn_p)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
215 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
216
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
217 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
218 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
219 "Prologue call to riscv_save_0 followed by noreturn call, "
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
220 "removing prologue.\n");
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
221 remove_insn (prologue);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
222 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
223
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
224 /* Entry point called from riscv_reorg to remove some unneeded calls to
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
225 the save and restore stubs. This should only be called when
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
226 -msave-restore is in use.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
227
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
228 We identify some simple cases where the function looks like this:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
229
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
230 call t0,__riscv_save_0
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
231 <other-code>
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
232 call foo
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
233 tail __riscv_restore_0
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
234
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
235 And transform it into something like this:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
236
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
237 <other-code>
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
238 tail foo
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
239
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
240 In the above examples, what can appear in <other-code> is pretty
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
241 restricted; only caller saved registers can be touched, this prevents
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
242 any additional calls (as they would write to 'ra'). */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
243
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
244 void
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
245 riscv_remove_unneeded_save_restore_calls (void)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
246 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
247 /* Will point to the first instruction of the function body, after the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
248 prologue end note. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
249 rtx_insn *body = NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
250
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
251 /* Should only be called with -msave-restore is in use. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
252 gcc_assert (TARGET_SAVE_RESTORE);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
253
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
254 /* Match the expected prologue and epilogue patterns. If either of these
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
255 fail to match then we abandon our attempt to optimize this function. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
256 rtx_insn *prologue_matched = riscv_sr_match_prologue (&body);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
257 if (prologue_matched == NULL || body == NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
258 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
259
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
260 rtx_insn *epilogue_matched = riscv_sr_match_epilogue ();
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
261 if (epilogue_matched == NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
262 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
263 check_for_no_return_call (prologue_matched);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
264 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
265 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
266
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
267 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
268 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
269 "Could be a candidate for save/restore removal\n");
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
270
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
271 /* We want to check which registers this function uses. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
272 df_analyze ();
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
273
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
274 int call_count = 0;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
275 bool good_use = true;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
276 int epilogue_count = 0;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
277
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
278 /* Now examine all of the instructions that make up this function, we're
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
279 looking for call instructions and also double checking register usage
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
280 while we're at it (see comments below). */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
281 basic_block bb;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
282 FOR_EACH_BB_FN (bb, cfun)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
283 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
284 rtx_insn *insn;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
285
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
286 FOR_BB_INSNS (bb, insn)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
287 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
288 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
289 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
290 "Block %d, Insn %d\n", bb->index, INSN_UID (insn));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
291
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
292 /* If we scan the epilogue we will fall foul of our register
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
293 usage check below (due to it's use of the return address), so
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
294 once we spot we're at the epilogue, just skip the rest of this
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
295 block. Scanning the prologue instructions again (if they
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
296 match the expected pattern) is harmless. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
297 if (NOTE_P (insn)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
298 && NOTE_KIND (insn) == NOTE_INSN_EPILOGUE_BEG)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
299 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
300 ++epilogue_count;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
301 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
302 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
303
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
304 if (!INSN_P (insn))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
305 continue;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
306
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
307 if (CALL_P (insn))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
308 ++call_count;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
309 else
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
310 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
311 df_ref use;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
312
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
313 FOR_EACH_INSN_USE (use, insn)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
314 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
315 /* If the function makes use of any registers that are
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
316 callee saved then we should be saving them in this
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
317 function, which would suggest that a call to the save
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
318 and restore functions is required. This would seem to
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
319 indicate that something has gone wrong above, as we
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
320 should only get here if we are saving zero registers.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
321
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
322 The one exception to this rule is the return address
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
323 register used within a call instruction. We can
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
324 optimize a single call within a function (by making it
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
325 a tail call), so we skip call instructions here. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
326 if (!call_used_regs[DF_REF_REGNO (use)])
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
327 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
328 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
329 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
330 "Found unsupported use of callee saved "
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
331 "register in instruction %d\n",
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
332 INSN_UID (insn));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
333 good_use = false;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
334 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
335 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
336 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
337 if (!good_use)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
338 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
339 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
340 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
341 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
342
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
343 /* If we used any registers that would indicate a need for a call to a
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
344 save/restore stub then don't optimize. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
345 if (!good_use)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
346 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
347
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
348 /* If this function has multiple epilogues, then for now we don't try to
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
349 optimize it. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
350 if (epilogue_count != 1)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
351 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
352
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
353 /* We can only optimize functions containing a single call, any more
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
354 would require us to add instructions to store the return address on
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
355 the stack (and restore it before we return). We could do this in the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
356 future, but for now we don't. A single call can be transformed into
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
357 a tail call reasonably easily. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
358 if (call_count > 1)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
359 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
360 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
361 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
362 "Found too many call instructions\n");
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
363 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
364 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
365
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
366 rtx_insn *epilogue_begin_note = PREV_INSN (epilogue_matched);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
367 gcc_assert (NOTE_P (epilogue_begin_note)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
368 && NOTE_KIND (epilogue_begin_note) == NOTE_INSN_EPILOGUE_BEG);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
369
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
370 df_finish_pass (false);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
371
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
372 /* Find the first instruction before the function epilogue. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
373 rtx_insn *insn_before_epilogue;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
374 for (insn_before_epilogue = PREV_INSN (epilogue_begin_note);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
375 NOTE_P (insn_before_epilogue);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
376 insn_before_epilogue = PREV_INSN (insn_before_epilogue))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
377 ;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
378
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
379 /* Leaf functions will not generate calls to the save/restore stubs, so
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
380 there's no need for this optimization there. We know this function
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
381 has no more than 1 call (checked above). To convert this single call
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
382 into a tail call we rely on the call being the last thing before the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
383 epilogue. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
384 if (GET_CODE (insn_before_epilogue) != CALL_INSN)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
385 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
386
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
387 /* The last instruction in this block, just before the epilogue is a
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
388 call. We can potentially change this call into a tail call. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
389 rtx_insn *call = insn_before_epilogue;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
390
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
391 /* Transform call in insn to a sibcall, this will only be done if the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
392 last thing in the function is a call. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
393 rtx callpat = PATTERN (call);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
394 gcc_assert (GET_CODE (callpat) == PARALLEL);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
395
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
396 /* Extract from CALLPAT the information we need to build the sibcall. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
397 rtx target_call = NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
398 rtx tmp_rtx = XVECEXP (callpat, 0, 0);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
399 rtx set_target = NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
400 switch (GET_CODE (tmp_rtx))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
401 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
402 case CALL:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
403 target_call = tmp_rtx;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
404 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
405
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
406 case SET:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
407 {
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
408 set_target = XEXP (tmp_rtx, 0);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
409 tmp_rtx = XEXP (tmp_rtx, 1);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
410 if (GET_CODE (tmp_rtx) != CALL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
411 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
412 target_call = tmp_rtx;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
413 break;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
414 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
415
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
416 default:
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
417 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
418 }
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
419
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
420 rtx target_mem = XEXP (target_call, 0);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
421 if (GET_CODE (target_mem) != MEM)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
422 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
423
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
424 rtx target = XEXP (target_mem, 0);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
425 if (GET_CODE (target) != SYMBOL_REF && GET_CODE (target) != REG)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
426 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
427
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
428 /* The sibcall instructions can only use a specific subset of
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
429 registers, we're about to (possibly) move a call through a
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
430 register from the function body and make it a sibcall. If we're
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
431 not using an appropriate register then we can't make this change.
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
432
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
433 Maybe in some future iteration we could actually scan the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
434 function, find a suitable sibcall register, and switch over the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
435 registers. But we don't do that yet. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
436 if (GET_CODE (target) == REG
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
437 && !SIBCALL_REG_P (REGNO (target)))
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
438 return;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
439
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
440 rtx sibcall = NULL;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
441 if (set_target != NULL)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
442 sibcall
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
443 = gen_sibcall_value_internal (set_target, target, const0_rtx);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
444 else
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
445 sibcall = gen_sibcall_internal (target, const0_rtx);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
446
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
447 rtx_insn *before_call = PREV_INSN (call);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
448 remove_insn (call);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
449 rtx_insn *insn = emit_call_insn_after_setloc (sibcall, before_call,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
450 INSN_LOCATION (call));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
451 REG_NOTES (insn) = REG_NOTES (call);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
452 SIBLING_CALL_P (insn) = 1;
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
453
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
454 /* Now update the prologue and epilogue to take account of the
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
455 changes within the function body. */
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
456 remove_insn (prologue_matched);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
457 remove_insn (NEXT_INSN (NEXT_INSN (NEXT_INSN (epilogue_matched))));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
458 remove_insn (NEXT_INSN (NEXT_INSN (epilogue_matched)));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
459 remove_insn (NEXT_INSN (epilogue_matched));
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
460 remove_insn (epilogue_matched);
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
461
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
462 if (dump_file)
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
463 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
464 "Save/restore successfully removed\n");
1830386684a0 gcc-9.2.0
anatofuz
parents:
diff changeset
465 }