annotate gcc/sanopt.c @ 158:494b0b89df80 default tip

...
author Shinji KONO <kono@ie.u-ryukyu.ac.jp>
date Mon, 25 May 2020 18:13:55 +0900
parents 1830386684a0
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
111
kono
parents:
diff changeset
1 /* Optimize and expand sanitizer functions.
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
2 Copyright (C) 2014-2020 Free Software Foundation, Inc.
111
kono
parents:
diff changeset
3 Contributed by Marek Polacek <polacek@redhat.com>
kono
parents:
diff changeset
4
kono
parents:
diff changeset
5 This file is part of GCC.
kono
parents:
diff changeset
6
kono
parents:
diff changeset
7 GCC is free software; you can redistribute it and/or modify it under
kono
parents:
diff changeset
8 the terms of the GNU General Public License as published by the Free
kono
parents:
diff changeset
9 Software Foundation; either version 3, or (at your option) any later
kono
parents:
diff changeset
10 version.
kono
parents:
diff changeset
11
kono
parents:
diff changeset
12 GCC is distributed in the hope that it will be useful, but WITHOUT ANY
kono
parents:
diff changeset
13 WARRANTY; without even the implied warranty of MERCHANTABILITY or
kono
parents:
diff changeset
14 FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
kono
parents:
diff changeset
15 for more details.
kono
parents:
diff changeset
16
kono
parents:
diff changeset
17 You should have received a copy of the GNU General Public License
kono
parents:
diff changeset
18 along with GCC; see the file COPYING3. If not see
kono
parents:
diff changeset
19 <http://www.gnu.org/licenses/>. */
kono
parents:
diff changeset
20
kono
parents:
diff changeset
21 #include "config.h"
kono
parents:
diff changeset
22 #include "system.h"
kono
parents:
diff changeset
23 #include "coretypes.h"
kono
parents:
diff changeset
24 #include "backend.h"
kono
parents:
diff changeset
25 #include "tree.h"
kono
parents:
diff changeset
26 #include "gimple.h"
kono
parents:
diff changeset
27 #include "ssa.h"
kono
parents:
diff changeset
28 #include "tree-pass.h"
kono
parents:
diff changeset
29 #include "tree-ssa-operands.h"
kono
parents:
diff changeset
30 #include "gimple-pretty-print.h"
kono
parents:
diff changeset
31 #include "fold-const.h"
kono
parents:
diff changeset
32 #include "gimple-iterator.h"
kono
parents:
diff changeset
33 #include "stringpool.h"
kono
parents:
diff changeset
34 #include "attribs.h"
kono
parents:
diff changeset
35 #include "asan.h"
kono
parents:
diff changeset
36 #include "ubsan.h"
kono
parents:
diff changeset
37 #include "tree-hash-traits.h"
kono
parents:
diff changeset
38 #include "gimple-ssa.h"
kono
parents:
diff changeset
39 #include "tree-phinodes.h"
kono
parents:
diff changeset
40 #include "ssa-iterators.h"
kono
parents:
diff changeset
41 #include "gimplify.h"
kono
parents:
diff changeset
42 #include "gimple-iterator.h"
kono
parents:
diff changeset
43 #include "gimple-walk.h"
kono
parents:
diff changeset
44 #include "cfghooks.h"
kono
parents:
diff changeset
45 #include "tree-dfa.h"
kono
parents:
diff changeset
46 #include "tree-ssa.h"
kono
parents:
diff changeset
47 #include "varasm.h"
kono
parents:
diff changeset
48
kono
parents:
diff changeset
49 /* This is used to carry information about basic blocks. It is
kono
parents:
diff changeset
50 attached to the AUX field of the standard CFG block. */
kono
parents:
diff changeset
51
kono
parents:
diff changeset
52 struct sanopt_info
kono
parents:
diff changeset
53 {
kono
parents:
diff changeset
54 /* True if this BB might call (directly or indirectly) free/munmap
kono
parents:
diff changeset
55 or similar operation. */
kono
parents:
diff changeset
56 bool has_freeing_call_p;
kono
parents:
diff changeset
57
kono
parents:
diff changeset
58 /* True if HAS_FREEING_CALL_P flag has been computed. */
kono
parents:
diff changeset
59 bool has_freeing_call_computed_p;
kono
parents:
diff changeset
60
kono
parents:
diff changeset
61 /* True if there is a block with HAS_FREEING_CALL_P flag set
kono
parents:
diff changeset
62 on any path between an immediate dominator of BB, denoted
kono
parents:
diff changeset
63 imm(BB), and BB. */
kono
parents:
diff changeset
64 bool imm_dom_path_with_freeing_call_p;
kono
parents:
diff changeset
65
kono
parents:
diff changeset
66 /* True if IMM_DOM_PATH_WITH_FREEING_CALL_P has been computed. */
kono
parents:
diff changeset
67 bool imm_dom_path_with_freeing_call_computed_p;
kono
parents:
diff changeset
68
kono
parents:
diff changeset
69 /* Number of possibly freeing calls encountered in this bb
kono
parents:
diff changeset
70 (so far). */
kono
parents:
diff changeset
71 uint64_t freeing_call_events;
kono
parents:
diff changeset
72
kono
parents:
diff changeset
73 /* True if BB is currently being visited during computation
kono
parents:
diff changeset
74 of IMM_DOM_PATH_WITH_FREEING_CALL_P flag. */
kono
parents:
diff changeset
75 bool being_visited_p;
kono
parents:
diff changeset
76
kono
parents:
diff changeset
77 /* True if this BB has been visited in the dominator walk. */
kono
parents:
diff changeset
78 bool visited_p;
kono
parents:
diff changeset
79 };
kono
parents:
diff changeset
80
kono
parents:
diff changeset
81 /* If T has a single definition of form T = T2, return T2. */
kono
parents:
diff changeset
82
kono
parents:
diff changeset
83 static tree
kono
parents:
diff changeset
84 maybe_get_single_definition (tree t)
kono
parents:
diff changeset
85 {
kono
parents:
diff changeset
86 if (TREE_CODE (t) == SSA_NAME)
kono
parents:
diff changeset
87 {
kono
parents:
diff changeset
88 gimple *g = SSA_NAME_DEF_STMT (t);
kono
parents:
diff changeset
89 if (gimple_assign_single_p (g))
kono
parents:
diff changeset
90 return gimple_assign_rhs1 (g);
kono
parents:
diff changeset
91 }
kono
parents:
diff changeset
92 return NULL_TREE;
kono
parents:
diff changeset
93 }
kono
parents:
diff changeset
94
kono
parents:
diff changeset
95 /* Tree triplet for vptr_check_map. */
kono
parents:
diff changeset
96 struct sanopt_tree_triplet
kono
parents:
diff changeset
97 {
kono
parents:
diff changeset
98 tree t1, t2, t3;
kono
parents:
diff changeset
99 };
kono
parents:
diff changeset
100
kono
parents:
diff changeset
101 /* Traits class for tree triplet hash maps below. */
kono
parents:
diff changeset
102
kono
parents:
diff changeset
103 struct sanopt_tree_triplet_hash : typed_noop_remove <sanopt_tree_triplet>
kono
parents:
diff changeset
104 {
kono
parents:
diff changeset
105 typedef sanopt_tree_triplet value_type;
kono
parents:
diff changeset
106 typedef sanopt_tree_triplet compare_type;
kono
parents:
diff changeset
107
kono
parents:
diff changeset
108 static hashval_t
kono
parents:
diff changeset
109 hash (const sanopt_tree_triplet &ref)
kono
parents:
diff changeset
110 {
kono
parents:
diff changeset
111 inchash::hash hstate (0);
kono
parents:
diff changeset
112 inchash::add_expr (ref.t1, hstate);
kono
parents:
diff changeset
113 inchash::add_expr (ref.t2, hstate);
kono
parents:
diff changeset
114 inchash::add_expr (ref.t3, hstate);
kono
parents:
diff changeset
115 return hstate.end ();
kono
parents:
diff changeset
116 }
kono
parents:
diff changeset
117
kono
parents:
diff changeset
118 static bool
kono
parents:
diff changeset
119 equal (const sanopt_tree_triplet &ref1, const sanopt_tree_triplet &ref2)
kono
parents:
diff changeset
120 {
kono
parents:
diff changeset
121 return operand_equal_p (ref1.t1, ref2.t1, 0)
kono
parents:
diff changeset
122 && operand_equal_p (ref1.t2, ref2.t2, 0)
kono
parents:
diff changeset
123 && operand_equal_p (ref1.t3, ref2.t3, 0);
kono
parents:
diff changeset
124 }
kono
parents:
diff changeset
125
kono
parents:
diff changeset
126 static void
kono
parents:
diff changeset
127 mark_deleted (sanopt_tree_triplet &ref)
kono
parents:
diff changeset
128 {
kono
parents:
diff changeset
129 ref.t1 = reinterpret_cast<tree> (1);
kono
parents:
diff changeset
130 }
kono
parents:
diff changeset
131
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
132 static const bool empty_zero_p = true;
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
133
111
kono
parents:
diff changeset
134 static void
kono
parents:
diff changeset
135 mark_empty (sanopt_tree_triplet &ref)
kono
parents:
diff changeset
136 {
kono
parents:
diff changeset
137 ref.t1 = NULL;
kono
parents:
diff changeset
138 }
kono
parents:
diff changeset
139
kono
parents:
diff changeset
140 static bool
kono
parents:
diff changeset
141 is_deleted (const sanopt_tree_triplet &ref)
kono
parents:
diff changeset
142 {
kono
parents:
diff changeset
143 return ref.t1 == reinterpret_cast<tree> (1);
kono
parents:
diff changeset
144 }
kono
parents:
diff changeset
145
kono
parents:
diff changeset
146 static bool
kono
parents:
diff changeset
147 is_empty (const sanopt_tree_triplet &ref)
kono
parents:
diff changeset
148 {
kono
parents:
diff changeset
149 return ref.t1 == NULL;
kono
parents:
diff changeset
150 }
kono
parents:
diff changeset
151 };
kono
parents:
diff changeset
152
kono
parents:
diff changeset
153 /* Tree couple for ptr_check_map. */
kono
parents:
diff changeset
154 struct sanopt_tree_couple
kono
parents:
diff changeset
155 {
kono
parents:
diff changeset
156 tree ptr;
kono
parents:
diff changeset
157 bool pos_p;
kono
parents:
diff changeset
158 };
kono
parents:
diff changeset
159
kono
parents:
diff changeset
160 /* Traits class for tree triplet hash maps below. */
kono
parents:
diff changeset
161
kono
parents:
diff changeset
162 struct sanopt_tree_couple_hash : typed_noop_remove <sanopt_tree_couple>
kono
parents:
diff changeset
163 {
kono
parents:
diff changeset
164 typedef sanopt_tree_couple value_type;
kono
parents:
diff changeset
165 typedef sanopt_tree_couple compare_type;
kono
parents:
diff changeset
166
kono
parents:
diff changeset
167 static hashval_t
kono
parents:
diff changeset
168 hash (const sanopt_tree_couple &ref)
kono
parents:
diff changeset
169 {
kono
parents:
diff changeset
170 inchash::hash hstate (0);
kono
parents:
diff changeset
171 inchash::add_expr (ref.ptr, hstate);
kono
parents:
diff changeset
172 hstate.add_int (ref.pos_p);
kono
parents:
diff changeset
173 return hstate.end ();
kono
parents:
diff changeset
174 }
kono
parents:
diff changeset
175
kono
parents:
diff changeset
176 static bool
kono
parents:
diff changeset
177 equal (const sanopt_tree_couple &ref1, const sanopt_tree_couple &ref2)
kono
parents:
diff changeset
178 {
kono
parents:
diff changeset
179 return operand_equal_p (ref1.ptr, ref2.ptr, 0)
kono
parents:
diff changeset
180 && ref1.pos_p == ref2.pos_p;
kono
parents:
diff changeset
181 }
kono
parents:
diff changeset
182
kono
parents:
diff changeset
183 static void
kono
parents:
diff changeset
184 mark_deleted (sanopt_tree_couple &ref)
kono
parents:
diff changeset
185 {
kono
parents:
diff changeset
186 ref.ptr = reinterpret_cast<tree> (1);
kono
parents:
diff changeset
187 }
kono
parents:
diff changeset
188
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
189 static const bool empty_zero_p = true;
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
190
111
kono
parents:
diff changeset
191 static void
kono
parents:
diff changeset
192 mark_empty (sanopt_tree_couple &ref)
kono
parents:
diff changeset
193 {
kono
parents:
diff changeset
194 ref.ptr = NULL;
kono
parents:
diff changeset
195 }
kono
parents:
diff changeset
196
kono
parents:
diff changeset
197 static bool
kono
parents:
diff changeset
198 is_deleted (const sanopt_tree_couple &ref)
kono
parents:
diff changeset
199 {
kono
parents:
diff changeset
200 return ref.ptr == reinterpret_cast<tree> (1);
kono
parents:
diff changeset
201 }
kono
parents:
diff changeset
202
kono
parents:
diff changeset
203 static bool
kono
parents:
diff changeset
204 is_empty (const sanopt_tree_couple &ref)
kono
parents:
diff changeset
205 {
kono
parents:
diff changeset
206 return ref.ptr == NULL;
kono
parents:
diff changeset
207 }
kono
parents:
diff changeset
208 };
kono
parents:
diff changeset
209
kono
parents:
diff changeset
210 /* This is used to carry various hash maps and variables used
kono
parents:
diff changeset
211 in sanopt_optimize_walker. */
kono
parents:
diff changeset
212
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
213 class sanopt_ctx
111
kono
parents:
diff changeset
214 {
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
215 public:
111
kono
parents:
diff changeset
216 /* This map maps a pointer (the first argument of UBSAN_NULL) to
kono
parents:
diff changeset
217 a vector of UBSAN_NULL call statements that check this pointer. */
kono
parents:
diff changeset
218 hash_map<tree, auto_vec<gimple *> > null_check_map;
kono
parents:
diff changeset
219
kono
parents:
diff changeset
220 /* This map maps a pointer (the second argument of ASAN_CHECK) to
kono
parents:
diff changeset
221 a vector of ASAN_CHECK call statements that check the access. */
kono
parents:
diff changeset
222 hash_map<tree_operand_hash, auto_vec<gimple *> > asan_check_map;
kono
parents:
diff changeset
223
kono
parents:
diff changeset
224 /* This map maps a tree triplet (the first, second and fourth argument
kono
parents:
diff changeset
225 of UBSAN_VPTR) to a vector of UBSAN_VPTR call statements that check
kono
parents:
diff changeset
226 that virtual table pointer. */
kono
parents:
diff changeset
227 hash_map<sanopt_tree_triplet_hash, auto_vec<gimple *> > vptr_check_map;
kono
parents:
diff changeset
228
kono
parents:
diff changeset
229 /* This map maps a couple (tree and boolean) to a vector of UBSAN_PTR
kono
parents:
diff changeset
230 call statements that check that pointer overflow. */
kono
parents:
diff changeset
231 hash_map<sanopt_tree_couple_hash, auto_vec<gimple *> > ptr_check_map;
kono
parents:
diff changeset
232
kono
parents:
diff changeset
233 /* Number of IFN_ASAN_CHECK statements. */
kono
parents:
diff changeset
234 int asan_num_accesses;
kono
parents:
diff changeset
235
kono
parents:
diff changeset
236 /* True when the current functions constains an ASAN_MARK. */
kono
parents:
diff changeset
237 bool contains_asan_mark;
kono
parents:
diff changeset
238 };
kono
parents:
diff changeset
239
kono
parents:
diff changeset
240 /* Return true if there might be any call to free/munmap operation
kono
parents:
diff changeset
241 on any path in between DOM (which should be imm(BB)) and BB. */
kono
parents:
diff changeset
242
kono
parents:
diff changeset
243 static bool
kono
parents:
diff changeset
244 imm_dom_path_with_freeing_call (basic_block bb, basic_block dom)
kono
parents:
diff changeset
245 {
kono
parents:
diff changeset
246 sanopt_info *info = (sanopt_info *) bb->aux;
kono
parents:
diff changeset
247 edge e;
kono
parents:
diff changeset
248 edge_iterator ei;
kono
parents:
diff changeset
249
kono
parents:
diff changeset
250 if (info->imm_dom_path_with_freeing_call_computed_p)
kono
parents:
diff changeset
251 return info->imm_dom_path_with_freeing_call_p;
kono
parents:
diff changeset
252
kono
parents:
diff changeset
253 info->being_visited_p = true;
kono
parents:
diff changeset
254
kono
parents:
diff changeset
255 FOR_EACH_EDGE (e, ei, bb->preds)
kono
parents:
diff changeset
256 {
kono
parents:
diff changeset
257 sanopt_info *pred_info = (sanopt_info *) e->src->aux;
kono
parents:
diff changeset
258
kono
parents:
diff changeset
259 if (e->src == dom)
kono
parents:
diff changeset
260 continue;
kono
parents:
diff changeset
261
kono
parents:
diff changeset
262 if ((pred_info->imm_dom_path_with_freeing_call_computed_p
kono
parents:
diff changeset
263 && pred_info->imm_dom_path_with_freeing_call_p)
kono
parents:
diff changeset
264 || (pred_info->has_freeing_call_computed_p
kono
parents:
diff changeset
265 && pred_info->has_freeing_call_p))
kono
parents:
diff changeset
266 {
kono
parents:
diff changeset
267 info->imm_dom_path_with_freeing_call_computed_p = true;
kono
parents:
diff changeset
268 info->imm_dom_path_with_freeing_call_p = true;
kono
parents:
diff changeset
269 info->being_visited_p = false;
kono
parents:
diff changeset
270 return true;
kono
parents:
diff changeset
271 }
kono
parents:
diff changeset
272 }
kono
parents:
diff changeset
273
kono
parents:
diff changeset
274 FOR_EACH_EDGE (e, ei, bb->preds)
kono
parents:
diff changeset
275 {
kono
parents:
diff changeset
276 sanopt_info *pred_info = (sanopt_info *) e->src->aux;
kono
parents:
diff changeset
277
kono
parents:
diff changeset
278 if (e->src == dom)
kono
parents:
diff changeset
279 continue;
kono
parents:
diff changeset
280
kono
parents:
diff changeset
281 if (pred_info->has_freeing_call_computed_p)
kono
parents:
diff changeset
282 continue;
kono
parents:
diff changeset
283
kono
parents:
diff changeset
284 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
285 for (gsi = gsi_start_bb (e->src); !gsi_end_p (gsi); gsi_next (&gsi))
kono
parents:
diff changeset
286 {
kono
parents:
diff changeset
287 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
288 gasm *asm_stmt;
kono
parents:
diff changeset
289
kono
parents:
diff changeset
290 if ((is_gimple_call (stmt) && !nonfreeing_call_p (stmt))
kono
parents:
diff changeset
291 || ((asm_stmt = dyn_cast <gasm *> (stmt))
kono
parents:
diff changeset
292 && (gimple_asm_clobbers_memory_p (asm_stmt)
kono
parents:
diff changeset
293 || gimple_asm_volatile_p (asm_stmt))))
kono
parents:
diff changeset
294 {
kono
parents:
diff changeset
295 pred_info->has_freeing_call_p = true;
kono
parents:
diff changeset
296 break;
kono
parents:
diff changeset
297 }
kono
parents:
diff changeset
298 }
kono
parents:
diff changeset
299
kono
parents:
diff changeset
300 pred_info->has_freeing_call_computed_p = true;
kono
parents:
diff changeset
301 if (pred_info->has_freeing_call_p)
kono
parents:
diff changeset
302 {
kono
parents:
diff changeset
303 info->imm_dom_path_with_freeing_call_computed_p = true;
kono
parents:
diff changeset
304 info->imm_dom_path_with_freeing_call_p = true;
kono
parents:
diff changeset
305 info->being_visited_p = false;
kono
parents:
diff changeset
306 return true;
kono
parents:
diff changeset
307 }
kono
parents:
diff changeset
308 }
kono
parents:
diff changeset
309
kono
parents:
diff changeset
310 FOR_EACH_EDGE (e, ei, bb->preds)
kono
parents:
diff changeset
311 {
kono
parents:
diff changeset
312 if (e->src == dom)
kono
parents:
diff changeset
313 continue;
kono
parents:
diff changeset
314
kono
parents:
diff changeset
315 basic_block src;
kono
parents:
diff changeset
316 for (src = e->src; src != dom; )
kono
parents:
diff changeset
317 {
kono
parents:
diff changeset
318 sanopt_info *pred_info = (sanopt_info *) src->aux;
kono
parents:
diff changeset
319 if (pred_info->being_visited_p)
kono
parents:
diff changeset
320 break;
kono
parents:
diff changeset
321 basic_block imm = get_immediate_dominator (CDI_DOMINATORS, src);
kono
parents:
diff changeset
322 if (imm_dom_path_with_freeing_call (src, imm))
kono
parents:
diff changeset
323 {
kono
parents:
diff changeset
324 info->imm_dom_path_with_freeing_call_computed_p = true;
kono
parents:
diff changeset
325 info->imm_dom_path_with_freeing_call_p = true;
kono
parents:
diff changeset
326 info->being_visited_p = false;
kono
parents:
diff changeset
327 return true;
kono
parents:
diff changeset
328 }
kono
parents:
diff changeset
329 src = imm;
kono
parents:
diff changeset
330 }
kono
parents:
diff changeset
331 }
kono
parents:
diff changeset
332
kono
parents:
diff changeset
333 info->imm_dom_path_with_freeing_call_computed_p = true;
kono
parents:
diff changeset
334 info->imm_dom_path_with_freeing_call_p = false;
kono
parents:
diff changeset
335 info->being_visited_p = false;
kono
parents:
diff changeset
336 return false;
kono
parents:
diff changeset
337 }
kono
parents:
diff changeset
338
kono
parents:
diff changeset
339 /* Get the first dominating check from the list of stored checks.
kono
parents:
diff changeset
340 Non-dominating checks are silently dropped. */
kono
parents:
diff changeset
341
kono
parents:
diff changeset
342 static gimple *
kono
parents:
diff changeset
343 maybe_get_dominating_check (auto_vec<gimple *> &v)
kono
parents:
diff changeset
344 {
kono
parents:
diff changeset
345 for (; !v.is_empty (); v.pop ())
kono
parents:
diff changeset
346 {
kono
parents:
diff changeset
347 gimple *g = v.last ();
kono
parents:
diff changeset
348 sanopt_info *si = (sanopt_info *) gimple_bb (g)->aux;
kono
parents:
diff changeset
349 if (!si->visited_p)
kono
parents:
diff changeset
350 /* At this point we shouldn't have any statements
kono
parents:
diff changeset
351 that aren't dominating the current BB. */
kono
parents:
diff changeset
352 return g;
kono
parents:
diff changeset
353 }
kono
parents:
diff changeset
354 return NULL;
kono
parents:
diff changeset
355 }
kono
parents:
diff changeset
356
kono
parents:
diff changeset
357 /* Optimize away redundant UBSAN_NULL calls. */
kono
parents:
diff changeset
358
kono
parents:
diff changeset
359 static bool
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
360 maybe_optimize_ubsan_null_ifn (class sanopt_ctx *ctx, gimple *stmt)
111
kono
parents:
diff changeset
361 {
kono
parents:
diff changeset
362 gcc_assert (gimple_call_num_args (stmt) == 3);
kono
parents:
diff changeset
363 tree ptr = gimple_call_arg (stmt, 0);
kono
parents:
diff changeset
364 tree cur_align = gimple_call_arg (stmt, 2);
kono
parents:
diff changeset
365 gcc_assert (TREE_CODE (cur_align) == INTEGER_CST);
kono
parents:
diff changeset
366 bool remove = false;
kono
parents:
diff changeset
367
kono
parents:
diff changeset
368 auto_vec<gimple *> &v = ctx->null_check_map.get_or_insert (ptr);
kono
parents:
diff changeset
369 gimple *g = maybe_get_dominating_check (v);
kono
parents:
diff changeset
370 if (!g)
kono
parents:
diff changeset
371 {
kono
parents:
diff changeset
372 /* For this PTR we don't have any UBSAN_NULL stmts recorded, so there's
kono
parents:
diff changeset
373 nothing to optimize yet. */
kono
parents:
diff changeset
374 v.safe_push (stmt);
kono
parents:
diff changeset
375 return false;
kono
parents:
diff changeset
376 }
kono
parents:
diff changeset
377
kono
parents:
diff changeset
378 /* We already have recorded a UBSAN_NULL check for this pointer. Perhaps we
kono
parents:
diff changeset
379 can drop this one. But only if this check doesn't specify stricter
kono
parents:
diff changeset
380 alignment. */
kono
parents:
diff changeset
381
kono
parents:
diff changeset
382 tree align = gimple_call_arg (g, 2);
kono
parents:
diff changeset
383 int kind = tree_to_shwi (gimple_call_arg (g, 1));
kono
parents:
diff changeset
384 /* If this is a NULL pointer check where we had segv anyway, we can
kono
parents:
diff changeset
385 remove it. */
kono
parents:
diff changeset
386 if (integer_zerop (align)
kono
parents:
diff changeset
387 && (kind == UBSAN_LOAD_OF
kono
parents:
diff changeset
388 || kind == UBSAN_STORE_OF
kono
parents:
diff changeset
389 || kind == UBSAN_MEMBER_ACCESS))
kono
parents:
diff changeset
390 remove = true;
kono
parents:
diff changeset
391 /* Otherwise remove the check in non-recovering mode, or if the
kono
parents:
diff changeset
392 stmts have same location. */
kono
parents:
diff changeset
393 else if (integer_zerop (align))
kono
parents:
diff changeset
394 remove = (flag_sanitize_recover & SANITIZE_NULL) == 0
kono
parents:
diff changeset
395 || flag_sanitize_undefined_trap_on_error
kono
parents:
diff changeset
396 || gimple_location (g) == gimple_location (stmt);
kono
parents:
diff changeset
397 else if (tree_int_cst_le (cur_align, align))
kono
parents:
diff changeset
398 remove = (flag_sanitize_recover & SANITIZE_ALIGNMENT) == 0
kono
parents:
diff changeset
399 || flag_sanitize_undefined_trap_on_error
kono
parents:
diff changeset
400 || gimple_location (g) == gimple_location (stmt);
kono
parents:
diff changeset
401
kono
parents:
diff changeset
402 if (!remove && gimple_bb (g) == gimple_bb (stmt)
kono
parents:
diff changeset
403 && tree_int_cst_compare (cur_align, align) == 0)
kono
parents:
diff changeset
404 v.pop ();
kono
parents:
diff changeset
405
kono
parents:
diff changeset
406 if (!remove)
kono
parents:
diff changeset
407 v.safe_push (stmt);
kono
parents:
diff changeset
408 return remove;
kono
parents:
diff changeset
409 }
kono
parents:
diff changeset
410
kono
parents:
diff changeset
411 /* Return true when pointer PTR for a given CUR_OFFSET is already sanitized
kono
parents:
diff changeset
412 in a given sanitization context CTX. */
kono
parents:
diff changeset
413
kono
parents:
diff changeset
414 static bool
kono
parents:
diff changeset
415 has_dominating_ubsan_ptr_check (sanopt_ctx *ctx, tree ptr,
kono
parents:
diff changeset
416 offset_int &cur_offset)
kono
parents:
diff changeset
417 {
kono
parents:
diff changeset
418 bool pos_p = !wi::neg_p (cur_offset);
kono
parents:
diff changeset
419 sanopt_tree_couple couple;
kono
parents:
diff changeset
420 couple.ptr = ptr;
kono
parents:
diff changeset
421 couple.pos_p = pos_p;
kono
parents:
diff changeset
422
kono
parents:
diff changeset
423 auto_vec<gimple *> &v = ctx->ptr_check_map.get_or_insert (couple);
kono
parents:
diff changeset
424 gimple *g = maybe_get_dominating_check (v);
kono
parents:
diff changeset
425 if (!g)
kono
parents:
diff changeset
426 return false;
kono
parents:
diff changeset
427
kono
parents:
diff changeset
428 /* We already have recorded a UBSAN_PTR check for this pointer. Perhaps we
kono
parents:
diff changeset
429 can drop this one. But only if this check doesn't specify larger offset.
kono
parents:
diff changeset
430 */
kono
parents:
diff changeset
431 tree offset = gimple_call_arg (g, 1);
kono
parents:
diff changeset
432 gcc_assert (TREE_CODE (offset) == INTEGER_CST);
kono
parents:
diff changeset
433 offset_int ooffset = wi::sext (wi::to_offset (offset), POINTER_SIZE);
kono
parents:
diff changeset
434
kono
parents:
diff changeset
435 if (pos_p)
kono
parents:
diff changeset
436 {
kono
parents:
diff changeset
437 if (wi::les_p (cur_offset, ooffset))
kono
parents:
diff changeset
438 return true;
kono
parents:
diff changeset
439 }
kono
parents:
diff changeset
440 else if (!pos_p && wi::les_p (ooffset, cur_offset))
kono
parents:
diff changeset
441 return true;
kono
parents:
diff changeset
442
kono
parents:
diff changeset
443 return false;
kono
parents:
diff changeset
444 }
kono
parents:
diff changeset
445
kono
parents:
diff changeset
446 /* Record UBSAN_PTR check of given context CTX. Register pointer PTR on
kono
parents:
diff changeset
447 a given OFFSET that it's handled by GIMPLE STMT. */
kono
parents:
diff changeset
448
kono
parents:
diff changeset
449 static void
kono
parents:
diff changeset
450 record_ubsan_ptr_check_stmt (sanopt_ctx *ctx, gimple *stmt, tree ptr,
kono
parents:
diff changeset
451 const offset_int &offset)
kono
parents:
diff changeset
452 {
kono
parents:
diff changeset
453 sanopt_tree_couple couple;
kono
parents:
diff changeset
454 couple.ptr = ptr;
kono
parents:
diff changeset
455 couple.pos_p = !wi::neg_p (offset);
kono
parents:
diff changeset
456
kono
parents:
diff changeset
457 auto_vec<gimple *> &v = ctx->ptr_check_map.get_or_insert (couple);
kono
parents:
diff changeset
458 v.safe_push (stmt);
kono
parents:
diff changeset
459 }
kono
parents:
diff changeset
460
kono
parents:
diff changeset
461 /* Optimize away redundant UBSAN_PTR calls. */
kono
parents:
diff changeset
462
kono
parents:
diff changeset
463 static bool
kono
parents:
diff changeset
464 maybe_optimize_ubsan_ptr_ifn (sanopt_ctx *ctx, gimple *stmt)
kono
parents:
diff changeset
465 {
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
466 poly_int64 bitsize, pbitpos;
111
kono
parents:
diff changeset
467 machine_mode mode;
kono
parents:
diff changeset
468 int volatilep = 0, reversep, unsignedp = 0;
kono
parents:
diff changeset
469 tree offset;
kono
parents:
diff changeset
470
kono
parents:
diff changeset
471 gcc_assert (gimple_call_num_args (stmt) == 2);
kono
parents:
diff changeset
472 tree ptr = gimple_call_arg (stmt, 0);
kono
parents:
diff changeset
473 tree off = gimple_call_arg (stmt, 1);
kono
parents:
diff changeset
474
kono
parents:
diff changeset
475 if (TREE_CODE (off) != INTEGER_CST)
kono
parents:
diff changeset
476 return false;
kono
parents:
diff changeset
477
kono
parents:
diff changeset
478 if (integer_zerop (off))
kono
parents:
diff changeset
479 return true;
kono
parents:
diff changeset
480
kono
parents:
diff changeset
481 offset_int cur_offset = wi::sext (wi::to_offset (off), POINTER_SIZE);
kono
parents:
diff changeset
482 if (has_dominating_ubsan_ptr_check (ctx, ptr, cur_offset))
kono
parents:
diff changeset
483 return true;
kono
parents:
diff changeset
484
kono
parents:
diff changeset
485 tree base = ptr;
kono
parents:
diff changeset
486 if (TREE_CODE (base) == ADDR_EXPR)
kono
parents:
diff changeset
487 {
kono
parents:
diff changeset
488 base = TREE_OPERAND (base, 0);
kono
parents:
diff changeset
489
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
490 HOST_WIDE_INT bitpos;
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
491 base = get_inner_reference (base, &bitsize, &pbitpos, &offset, &mode,
111
kono
parents:
diff changeset
492 &unsignedp, &reversep, &volatilep);
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
493 if ((offset == NULL_TREE || TREE_CODE (offset) == INTEGER_CST)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
494 && DECL_P (base)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
495 && !DECL_REGISTER (base)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
496 && pbitpos.is_constant (&bitpos))
111
kono
parents:
diff changeset
497 {
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
498 offset_int expr_offset;
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
499 if (offset)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
500 expr_offset = wi::to_offset (offset) + bitpos / BITS_PER_UNIT;
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
501 else
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
502 expr_offset = bitpos / BITS_PER_UNIT;
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
503 expr_offset = wi::sext (expr_offset, POINTER_SIZE);
111
kono
parents:
diff changeset
504 offset_int total_offset = expr_offset + cur_offset;
kono
parents:
diff changeset
505 if (total_offset != wi::sext (total_offset, POINTER_SIZE))
kono
parents:
diff changeset
506 {
kono
parents:
diff changeset
507 record_ubsan_ptr_check_stmt (ctx, stmt, ptr, cur_offset);
kono
parents:
diff changeset
508 return false;
kono
parents:
diff changeset
509 }
kono
parents:
diff changeset
510
kono
parents:
diff changeset
511 /* If BASE is a fixed size automatic variable or
kono
parents:
diff changeset
512 global variable defined in the current TU, we don't have
kono
parents:
diff changeset
513 to instrument anything if offset is within address
kono
parents:
diff changeset
514 of the variable. */
kono
parents:
diff changeset
515 if ((VAR_P (base)
kono
parents:
diff changeset
516 || TREE_CODE (base) == PARM_DECL
kono
parents:
diff changeset
517 || TREE_CODE (base) == RESULT_DECL)
kono
parents:
diff changeset
518 && DECL_SIZE_UNIT (base)
kono
parents:
diff changeset
519 && TREE_CODE (DECL_SIZE_UNIT (base)) == INTEGER_CST
kono
parents:
diff changeset
520 && (!is_global_var (base) || decl_binds_to_current_def_p (base)))
kono
parents:
diff changeset
521 {
kono
parents:
diff changeset
522 offset_int base_size = wi::to_offset (DECL_SIZE_UNIT (base));
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
523 if (!wi::neg_p (expr_offset)
111
kono
parents:
diff changeset
524 && wi::les_p (total_offset, base_size))
kono
parents:
diff changeset
525 {
kono
parents:
diff changeset
526 if (!wi::neg_p (total_offset)
kono
parents:
diff changeset
527 && wi::les_p (total_offset, base_size))
kono
parents:
diff changeset
528 return true;
kono
parents:
diff changeset
529 }
kono
parents:
diff changeset
530 }
kono
parents:
diff changeset
531
kono
parents:
diff changeset
532 /* Following expression: UBSAN_PTR (&MEM_REF[ptr + x], y) can be
kono
parents:
diff changeset
533 handled as follows:
kono
parents:
diff changeset
534
kono
parents:
diff changeset
535 1) sign (x) == sign (y), then check for dominating check of (x + y)
kono
parents:
diff changeset
536 2) sign (x) != sign (y), then first check if we have a dominating
kono
parents:
diff changeset
537 check for ptr + x. If so, then we have 2 situations:
kono
parents:
diff changeset
538 a) sign (x) == sign (x + y), here we are done, example:
kono
parents:
diff changeset
539 UBSAN_PTR (&MEM_REF[ptr + 100], -50)
kono
parents:
diff changeset
540 b) check for dominating check of ptr + x + y.
kono
parents:
diff changeset
541 */
kono
parents:
diff changeset
542
kono
parents:
diff changeset
543 bool sign_cur_offset = !wi::neg_p (cur_offset);
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
544 bool sign_expr_offset = !wi::neg_p (expr_offset);
111
kono
parents:
diff changeset
545
kono
parents:
diff changeset
546 tree base_addr
kono
parents:
diff changeset
547 = build1 (ADDR_EXPR, build_pointer_type (TREE_TYPE (base)), base);
kono
parents:
diff changeset
548
kono
parents:
diff changeset
549 bool add = false;
kono
parents:
diff changeset
550 if (sign_cur_offset == sign_expr_offset)
kono
parents:
diff changeset
551 {
kono
parents:
diff changeset
552 if (has_dominating_ubsan_ptr_check (ctx, base_addr, total_offset))
kono
parents:
diff changeset
553 return true;
kono
parents:
diff changeset
554 else
kono
parents:
diff changeset
555 add = true;
kono
parents:
diff changeset
556 }
kono
parents:
diff changeset
557 else
kono
parents:
diff changeset
558 {
kono
parents:
diff changeset
559 if (!has_dominating_ubsan_ptr_check (ctx, base_addr, expr_offset))
kono
parents:
diff changeset
560 ; /* Don't record base_addr + expr_offset, it's not a guarding
kono
parents:
diff changeset
561 check. */
kono
parents:
diff changeset
562 else
kono
parents:
diff changeset
563 {
kono
parents:
diff changeset
564 bool sign_total_offset = !wi::neg_p (total_offset);
kono
parents:
diff changeset
565 if (sign_expr_offset == sign_total_offset)
kono
parents:
diff changeset
566 return true;
kono
parents:
diff changeset
567 else
kono
parents:
diff changeset
568 {
kono
parents:
diff changeset
569 if (has_dominating_ubsan_ptr_check (ctx, base_addr,
kono
parents:
diff changeset
570 total_offset))
kono
parents:
diff changeset
571 return true;
kono
parents:
diff changeset
572 else
kono
parents:
diff changeset
573 add = true;
kono
parents:
diff changeset
574 }
kono
parents:
diff changeset
575 }
kono
parents:
diff changeset
576 }
kono
parents:
diff changeset
577
kono
parents:
diff changeset
578 /* Record a new dominating check for base_addr + total_offset. */
kono
parents:
diff changeset
579 if (add && !operand_equal_p (base, base_addr, 0))
kono
parents:
diff changeset
580 record_ubsan_ptr_check_stmt (ctx, stmt, base_addr,
kono
parents:
diff changeset
581 total_offset);
kono
parents:
diff changeset
582 }
kono
parents:
diff changeset
583 }
kono
parents:
diff changeset
584
kono
parents:
diff changeset
585 /* For this PTR we don't have any UBSAN_PTR stmts recorded, so there's
kono
parents:
diff changeset
586 nothing to optimize yet. */
kono
parents:
diff changeset
587 record_ubsan_ptr_check_stmt (ctx, stmt, ptr, cur_offset);
kono
parents:
diff changeset
588
kono
parents:
diff changeset
589 return false;
kono
parents:
diff changeset
590 }
kono
parents:
diff changeset
591
kono
parents:
diff changeset
592 /* Optimize away redundant UBSAN_VPTR calls. The second argument
kono
parents:
diff changeset
593 is the value loaded from the virtual table, so rely on FRE to find out
kono
parents:
diff changeset
594 when we can actually optimize. */
kono
parents:
diff changeset
595
kono
parents:
diff changeset
596 static bool
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
597 maybe_optimize_ubsan_vptr_ifn (class sanopt_ctx *ctx, gimple *stmt)
111
kono
parents:
diff changeset
598 {
kono
parents:
diff changeset
599 gcc_assert (gimple_call_num_args (stmt) == 5);
kono
parents:
diff changeset
600 sanopt_tree_triplet triplet;
kono
parents:
diff changeset
601 triplet.t1 = gimple_call_arg (stmt, 0);
kono
parents:
diff changeset
602 triplet.t2 = gimple_call_arg (stmt, 1);
kono
parents:
diff changeset
603 triplet.t3 = gimple_call_arg (stmt, 3);
kono
parents:
diff changeset
604
kono
parents:
diff changeset
605 auto_vec<gimple *> &v = ctx->vptr_check_map.get_or_insert (triplet);
kono
parents:
diff changeset
606 gimple *g = maybe_get_dominating_check (v);
kono
parents:
diff changeset
607 if (!g)
kono
parents:
diff changeset
608 {
kono
parents:
diff changeset
609 /* For this PTR we don't have any UBSAN_VPTR stmts recorded, so there's
kono
parents:
diff changeset
610 nothing to optimize yet. */
kono
parents:
diff changeset
611 v.safe_push (stmt);
kono
parents:
diff changeset
612 return false;
kono
parents:
diff changeset
613 }
kono
parents:
diff changeset
614
kono
parents:
diff changeset
615 return true;
kono
parents:
diff changeset
616 }
kono
parents:
diff changeset
617
kono
parents:
diff changeset
618 /* Returns TRUE if ASan check of length LEN in block BB can be removed
kono
parents:
diff changeset
619 if preceded by checks in V. */
kono
parents:
diff changeset
620
kono
parents:
diff changeset
621 static bool
kono
parents:
diff changeset
622 can_remove_asan_check (auto_vec<gimple *> &v, tree len, basic_block bb)
kono
parents:
diff changeset
623 {
kono
parents:
diff changeset
624 unsigned int i;
kono
parents:
diff changeset
625 gimple *g;
kono
parents:
diff changeset
626 gimple *to_pop = NULL;
kono
parents:
diff changeset
627 bool remove = false;
kono
parents:
diff changeset
628 basic_block last_bb = bb;
kono
parents:
diff changeset
629 bool cleanup = false;
kono
parents:
diff changeset
630
kono
parents:
diff changeset
631 FOR_EACH_VEC_ELT_REVERSE (v, i, g)
kono
parents:
diff changeset
632 {
kono
parents:
diff changeset
633 basic_block gbb = gimple_bb (g);
kono
parents:
diff changeset
634 sanopt_info *si = (sanopt_info *) gbb->aux;
kono
parents:
diff changeset
635 if (gimple_uid (g) < si->freeing_call_events)
kono
parents:
diff changeset
636 {
kono
parents:
diff changeset
637 /* If there is a potentially freeing call after g in gbb, we should
kono
parents:
diff changeset
638 remove it from the vector, can't use in optimization. */
kono
parents:
diff changeset
639 cleanup = true;
kono
parents:
diff changeset
640 continue;
kono
parents:
diff changeset
641 }
kono
parents:
diff changeset
642
kono
parents:
diff changeset
643 tree glen = gimple_call_arg (g, 2);
kono
parents:
diff changeset
644 gcc_assert (TREE_CODE (glen) == INTEGER_CST);
kono
parents:
diff changeset
645
kono
parents:
diff changeset
646 /* If we've checked only smaller length than we want to check now,
kono
parents:
diff changeset
647 we can't remove the current stmt. If g is in the same basic block,
kono
parents:
diff changeset
648 we want to remove it though, as the current stmt is better. */
kono
parents:
diff changeset
649 if (tree_int_cst_lt (glen, len))
kono
parents:
diff changeset
650 {
kono
parents:
diff changeset
651 if (gbb == bb)
kono
parents:
diff changeset
652 {
kono
parents:
diff changeset
653 to_pop = g;
kono
parents:
diff changeset
654 cleanup = true;
kono
parents:
diff changeset
655 }
kono
parents:
diff changeset
656 continue;
kono
parents:
diff changeset
657 }
kono
parents:
diff changeset
658
kono
parents:
diff changeset
659 while (last_bb != gbb)
kono
parents:
diff changeset
660 {
kono
parents:
diff changeset
661 /* Paths from last_bb to bb have been checked before.
kono
parents:
diff changeset
662 gbb is necessarily a dominator of last_bb, but not necessarily
kono
parents:
diff changeset
663 immediate dominator. */
kono
parents:
diff changeset
664 if (((sanopt_info *) last_bb->aux)->freeing_call_events)
kono
parents:
diff changeset
665 break;
kono
parents:
diff changeset
666
kono
parents:
diff changeset
667 basic_block imm = get_immediate_dominator (CDI_DOMINATORS, last_bb);
kono
parents:
diff changeset
668 gcc_assert (imm);
kono
parents:
diff changeset
669 if (imm_dom_path_with_freeing_call (last_bb, imm))
kono
parents:
diff changeset
670 break;
kono
parents:
diff changeset
671
kono
parents:
diff changeset
672 last_bb = imm;
kono
parents:
diff changeset
673 }
kono
parents:
diff changeset
674 if (last_bb == gbb)
kono
parents:
diff changeset
675 remove = true;
kono
parents:
diff changeset
676 break;
kono
parents:
diff changeset
677 }
kono
parents:
diff changeset
678
kono
parents:
diff changeset
679 if (cleanup)
kono
parents:
diff changeset
680 {
kono
parents:
diff changeset
681 unsigned int j = 0, l = v.length ();
kono
parents:
diff changeset
682 for (i = 0; i < l; i++)
kono
parents:
diff changeset
683 if (v[i] != to_pop
kono
parents:
diff changeset
684 && (gimple_uid (v[i])
kono
parents:
diff changeset
685 == ((sanopt_info *)
kono
parents:
diff changeset
686 gimple_bb (v[i])->aux)->freeing_call_events))
kono
parents:
diff changeset
687 {
kono
parents:
diff changeset
688 if (i != j)
kono
parents:
diff changeset
689 v[j] = v[i];
kono
parents:
diff changeset
690 j++;
kono
parents:
diff changeset
691 }
kono
parents:
diff changeset
692 v.truncate (j);
kono
parents:
diff changeset
693 }
kono
parents:
diff changeset
694
kono
parents:
diff changeset
695 return remove;
kono
parents:
diff changeset
696 }
kono
parents:
diff changeset
697
kono
parents:
diff changeset
698 /* Optimize away redundant ASAN_CHECK calls. */
kono
parents:
diff changeset
699
kono
parents:
diff changeset
700 static bool
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
701 maybe_optimize_asan_check_ifn (class sanopt_ctx *ctx, gimple *stmt)
111
kono
parents:
diff changeset
702 {
kono
parents:
diff changeset
703 gcc_assert (gimple_call_num_args (stmt) == 4);
kono
parents:
diff changeset
704 tree ptr = gimple_call_arg (stmt, 1);
kono
parents:
diff changeset
705 tree len = gimple_call_arg (stmt, 2);
kono
parents:
diff changeset
706 basic_block bb = gimple_bb (stmt);
kono
parents:
diff changeset
707 sanopt_info *info = (sanopt_info *) bb->aux;
kono
parents:
diff changeset
708
kono
parents:
diff changeset
709 if (TREE_CODE (len) != INTEGER_CST)
kono
parents:
diff changeset
710 return false;
kono
parents:
diff changeset
711 if (integer_zerop (len))
kono
parents:
diff changeset
712 return false;
kono
parents:
diff changeset
713
kono
parents:
diff changeset
714 gimple_set_uid (stmt, info->freeing_call_events);
kono
parents:
diff changeset
715
kono
parents:
diff changeset
716 auto_vec<gimple *> *ptr_checks = &ctx->asan_check_map.get_or_insert (ptr);
kono
parents:
diff changeset
717
kono
parents:
diff changeset
718 tree base_addr = maybe_get_single_definition (ptr);
kono
parents:
diff changeset
719 auto_vec<gimple *> *base_checks = NULL;
kono
parents:
diff changeset
720 if (base_addr)
kono
parents:
diff changeset
721 {
kono
parents:
diff changeset
722 base_checks = &ctx->asan_check_map.get_or_insert (base_addr);
kono
parents:
diff changeset
723 /* Original pointer might have been invalidated. */
kono
parents:
diff changeset
724 ptr_checks = ctx->asan_check_map.get (ptr);
kono
parents:
diff changeset
725 }
kono
parents:
diff changeset
726
kono
parents:
diff changeset
727 gimple *g = maybe_get_dominating_check (*ptr_checks);
kono
parents:
diff changeset
728 gimple *g2 = NULL;
kono
parents:
diff changeset
729
kono
parents:
diff changeset
730 if (base_checks)
kono
parents:
diff changeset
731 /* Try with base address as well. */
kono
parents:
diff changeset
732 g2 = maybe_get_dominating_check (*base_checks);
kono
parents:
diff changeset
733
kono
parents:
diff changeset
734 if (g == NULL && g2 == NULL)
kono
parents:
diff changeset
735 {
kono
parents:
diff changeset
736 /* For this PTR we don't have any ASAN_CHECK stmts recorded, so there's
kono
parents:
diff changeset
737 nothing to optimize yet. */
kono
parents:
diff changeset
738 ptr_checks->safe_push (stmt);
kono
parents:
diff changeset
739 if (base_checks)
kono
parents:
diff changeset
740 base_checks->safe_push (stmt);
kono
parents:
diff changeset
741 return false;
kono
parents:
diff changeset
742 }
kono
parents:
diff changeset
743
kono
parents:
diff changeset
744 bool remove = false;
kono
parents:
diff changeset
745
kono
parents:
diff changeset
746 if (ptr_checks)
kono
parents:
diff changeset
747 remove = can_remove_asan_check (*ptr_checks, len, bb);
kono
parents:
diff changeset
748
kono
parents:
diff changeset
749 if (!remove && base_checks)
kono
parents:
diff changeset
750 /* Try with base address as well. */
kono
parents:
diff changeset
751 remove = can_remove_asan_check (*base_checks, len, bb);
kono
parents:
diff changeset
752
kono
parents:
diff changeset
753 if (!remove)
kono
parents:
diff changeset
754 {
kono
parents:
diff changeset
755 ptr_checks->safe_push (stmt);
kono
parents:
diff changeset
756 if (base_checks)
kono
parents:
diff changeset
757 base_checks->safe_push (stmt);
kono
parents:
diff changeset
758 }
kono
parents:
diff changeset
759
kono
parents:
diff changeset
760 return remove;
kono
parents:
diff changeset
761 }
kono
parents:
diff changeset
762
kono
parents:
diff changeset
763 /* Try to optimize away redundant UBSAN_NULL and ASAN_CHECK calls.
kono
parents:
diff changeset
764
kono
parents:
diff changeset
765 We walk blocks in the CFG via a depth first search of the dominator
kono
parents:
diff changeset
766 tree; we push unique UBSAN_NULL or ASAN_CHECK statements into a vector
kono
parents:
diff changeset
767 in the NULL_CHECK_MAP or ASAN_CHECK_MAP hash maps as we enter the
kono
parents:
diff changeset
768 blocks. When leaving a block, we mark the block as visited; then
kono
parents:
diff changeset
769 when checking the statements in the vector, we ignore statements that
kono
parents:
diff changeset
770 are coming from already visited blocks, because these cannot dominate
kono
parents:
diff changeset
771 anything anymore. CTX is a sanopt context. */
kono
parents:
diff changeset
772
kono
parents:
diff changeset
773 static void
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
774 sanopt_optimize_walker (basic_block bb, class sanopt_ctx *ctx)
111
kono
parents:
diff changeset
775 {
kono
parents:
diff changeset
776 basic_block son;
kono
parents:
diff changeset
777 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
778 sanopt_info *info = (sanopt_info *) bb->aux;
kono
parents:
diff changeset
779 bool asan_check_optimize = (flag_sanitize & SANITIZE_ADDRESS) != 0;
kono
parents:
diff changeset
780
kono
parents:
diff changeset
781 for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi);)
kono
parents:
diff changeset
782 {
kono
parents:
diff changeset
783 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
784 bool remove = false;
kono
parents:
diff changeset
785
kono
parents:
diff changeset
786 if (!is_gimple_call (stmt))
kono
parents:
diff changeset
787 {
kono
parents:
diff changeset
788 /* Handle asm volatile or asm with "memory" clobber
kono
parents:
diff changeset
789 the same as potentionally freeing call. */
kono
parents:
diff changeset
790 gasm *asm_stmt = dyn_cast <gasm *> (stmt);
kono
parents:
diff changeset
791 if (asm_stmt
kono
parents:
diff changeset
792 && asan_check_optimize
kono
parents:
diff changeset
793 && (gimple_asm_clobbers_memory_p (asm_stmt)
kono
parents:
diff changeset
794 || gimple_asm_volatile_p (asm_stmt)))
kono
parents:
diff changeset
795 info->freeing_call_events++;
kono
parents:
diff changeset
796 gsi_next (&gsi);
kono
parents:
diff changeset
797 continue;
kono
parents:
diff changeset
798 }
kono
parents:
diff changeset
799
kono
parents:
diff changeset
800 if (asan_check_optimize && !nonfreeing_call_p (stmt))
kono
parents:
diff changeset
801 info->freeing_call_events++;
kono
parents:
diff changeset
802
kono
parents:
diff changeset
803 /* If __asan_before_dynamic_init ("module"); is followed by
kono
parents:
diff changeset
804 __asan_after_dynamic_init (); without intervening memory loads/stores,
kono
parents:
diff changeset
805 there is nothing to guard, so optimize both away. */
kono
parents:
diff changeset
806 if (asan_check_optimize
kono
parents:
diff changeset
807 && gimple_call_builtin_p (stmt, BUILT_IN_ASAN_BEFORE_DYNAMIC_INIT))
kono
parents:
diff changeset
808 {
kono
parents:
diff changeset
809 use_operand_p use;
kono
parents:
diff changeset
810 gimple *use_stmt;
kono
parents:
diff changeset
811 if (single_imm_use (gimple_vdef (stmt), &use, &use_stmt))
kono
parents:
diff changeset
812 {
kono
parents:
diff changeset
813 if (is_gimple_call (use_stmt)
kono
parents:
diff changeset
814 && gimple_call_builtin_p (use_stmt,
kono
parents:
diff changeset
815 BUILT_IN_ASAN_AFTER_DYNAMIC_INIT))
kono
parents:
diff changeset
816 {
kono
parents:
diff changeset
817 unlink_stmt_vdef (use_stmt);
kono
parents:
diff changeset
818 gimple_stmt_iterator gsi2 = gsi_for_stmt (use_stmt);
kono
parents:
diff changeset
819 gsi_remove (&gsi2, true);
kono
parents:
diff changeset
820 remove = true;
kono
parents:
diff changeset
821 }
kono
parents:
diff changeset
822 }
kono
parents:
diff changeset
823 }
kono
parents:
diff changeset
824
kono
parents:
diff changeset
825 if (gimple_call_internal_p (stmt))
kono
parents:
diff changeset
826 switch (gimple_call_internal_fn (stmt))
kono
parents:
diff changeset
827 {
kono
parents:
diff changeset
828 case IFN_UBSAN_NULL:
kono
parents:
diff changeset
829 remove = maybe_optimize_ubsan_null_ifn (ctx, stmt);
kono
parents:
diff changeset
830 break;
kono
parents:
diff changeset
831 case IFN_UBSAN_VPTR:
kono
parents:
diff changeset
832 remove = maybe_optimize_ubsan_vptr_ifn (ctx, stmt);
kono
parents:
diff changeset
833 break;
kono
parents:
diff changeset
834 case IFN_UBSAN_PTR:
kono
parents:
diff changeset
835 remove = maybe_optimize_ubsan_ptr_ifn (ctx, stmt);
kono
parents:
diff changeset
836 break;
kono
parents:
diff changeset
837 case IFN_ASAN_CHECK:
kono
parents:
diff changeset
838 if (asan_check_optimize)
kono
parents:
diff changeset
839 remove = maybe_optimize_asan_check_ifn (ctx, stmt);
kono
parents:
diff changeset
840 if (!remove)
kono
parents:
diff changeset
841 ctx->asan_num_accesses++;
kono
parents:
diff changeset
842 break;
kono
parents:
diff changeset
843 case IFN_ASAN_MARK:
kono
parents:
diff changeset
844 ctx->contains_asan_mark = true;
kono
parents:
diff changeset
845 break;
kono
parents:
diff changeset
846 default:
kono
parents:
diff changeset
847 break;
kono
parents:
diff changeset
848 }
kono
parents:
diff changeset
849
kono
parents:
diff changeset
850 if (remove)
kono
parents:
diff changeset
851 {
kono
parents:
diff changeset
852 /* Drop this check. */
kono
parents:
diff changeset
853 if (dump_file && (dump_flags & TDF_DETAILS))
kono
parents:
diff changeset
854 {
kono
parents:
diff changeset
855 fprintf (dump_file, "Optimizing out: ");
kono
parents:
diff changeset
856 print_gimple_stmt (dump_file, stmt, 0, dump_flags);
kono
parents:
diff changeset
857 }
kono
parents:
diff changeset
858 unlink_stmt_vdef (stmt);
kono
parents:
diff changeset
859 gsi_remove (&gsi, true);
kono
parents:
diff changeset
860 }
kono
parents:
diff changeset
861 else
kono
parents:
diff changeset
862 {
kono
parents:
diff changeset
863 if (dump_file && (dump_flags & TDF_DETAILS))
kono
parents:
diff changeset
864 {
kono
parents:
diff changeset
865 fprintf (dump_file, "Leaving: ");
kono
parents:
diff changeset
866 print_gimple_stmt (dump_file, stmt, 0, dump_flags);
kono
parents:
diff changeset
867 }
kono
parents:
diff changeset
868
kono
parents:
diff changeset
869 gsi_next (&gsi);
kono
parents:
diff changeset
870 }
kono
parents:
diff changeset
871 }
kono
parents:
diff changeset
872
kono
parents:
diff changeset
873 if (asan_check_optimize)
kono
parents:
diff changeset
874 {
kono
parents:
diff changeset
875 info->has_freeing_call_p = info->freeing_call_events != 0;
kono
parents:
diff changeset
876 info->has_freeing_call_computed_p = true;
kono
parents:
diff changeset
877 }
kono
parents:
diff changeset
878
kono
parents:
diff changeset
879 for (son = first_dom_son (CDI_DOMINATORS, bb);
kono
parents:
diff changeset
880 son;
kono
parents:
diff changeset
881 son = next_dom_son (CDI_DOMINATORS, son))
kono
parents:
diff changeset
882 sanopt_optimize_walker (son, ctx);
kono
parents:
diff changeset
883
kono
parents:
diff changeset
884 /* We're leaving this BB, so mark it to that effect. */
kono
parents:
diff changeset
885 info->visited_p = true;
kono
parents:
diff changeset
886 }
kono
parents:
diff changeset
887
kono
parents:
diff changeset
888 /* Try to remove redundant sanitizer checks in function FUN. */
kono
parents:
diff changeset
889
kono
parents:
diff changeset
890 static int
kono
parents:
diff changeset
891 sanopt_optimize (function *fun, bool *contains_asan_mark)
kono
parents:
diff changeset
892 {
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
893 class sanopt_ctx ctx;
111
kono
parents:
diff changeset
894 ctx.asan_num_accesses = 0;
kono
parents:
diff changeset
895 ctx.contains_asan_mark = false;
kono
parents:
diff changeset
896
kono
parents:
diff changeset
897 /* Set up block info for each basic block. */
kono
parents:
diff changeset
898 alloc_aux_for_blocks (sizeof (sanopt_info));
kono
parents:
diff changeset
899
kono
parents:
diff changeset
900 /* We're going to do a dominator walk, so ensure that we have
kono
parents:
diff changeset
901 dominance information. */
kono
parents:
diff changeset
902 calculate_dominance_info (CDI_DOMINATORS);
kono
parents:
diff changeset
903
kono
parents:
diff changeset
904 /* Recursively walk the dominator tree optimizing away
kono
parents:
diff changeset
905 redundant checks. */
kono
parents:
diff changeset
906 sanopt_optimize_walker (ENTRY_BLOCK_PTR_FOR_FN (fun), &ctx);
kono
parents:
diff changeset
907
kono
parents:
diff changeset
908 free_aux_for_blocks ();
kono
parents:
diff changeset
909
kono
parents:
diff changeset
910 *contains_asan_mark = ctx.contains_asan_mark;
kono
parents:
diff changeset
911 return ctx.asan_num_accesses;
kono
parents:
diff changeset
912 }
kono
parents:
diff changeset
913
kono
parents:
diff changeset
914 /* Perform optimization of sanitize functions. */
kono
parents:
diff changeset
915
kono
parents:
diff changeset
916 namespace {
kono
parents:
diff changeset
917
kono
parents:
diff changeset
918 const pass_data pass_data_sanopt =
kono
parents:
diff changeset
919 {
kono
parents:
diff changeset
920 GIMPLE_PASS, /* type */
kono
parents:
diff changeset
921 "sanopt", /* name */
kono
parents:
diff changeset
922 OPTGROUP_NONE, /* optinfo_flags */
kono
parents:
diff changeset
923 TV_NONE, /* tv_id */
kono
parents:
diff changeset
924 ( PROP_ssa | PROP_cfg | PROP_gimple_leh ), /* properties_required */
kono
parents:
diff changeset
925 0, /* properties_provided */
kono
parents:
diff changeset
926 0, /* properties_destroyed */
kono
parents:
diff changeset
927 0, /* todo_flags_start */
kono
parents:
diff changeset
928 TODO_update_ssa, /* todo_flags_finish */
kono
parents:
diff changeset
929 };
kono
parents:
diff changeset
930
kono
parents:
diff changeset
931 class pass_sanopt : public gimple_opt_pass
kono
parents:
diff changeset
932 {
kono
parents:
diff changeset
933 public:
kono
parents:
diff changeset
934 pass_sanopt (gcc::context *ctxt)
kono
parents:
diff changeset
935 : gimple_opt_pass (pass_data_sanopt, ctxt)
kono
parents:
diff changeset
936 {}
kono
parents:
diff changeset
937
kono
parents:
diff changeset
938 /* opt_pass methods: */
kono
parents:
diff changeset
939 virtual bool gate (function *) { return flag_sanitize; }
kono
parents:
diff changeset
940 virtual unsigned int execute (function *);
kono
parents:
diff changeset
941
kono
parents:
diff changeset
942 }; // class pass_sanopt
kono
parents:
diff changeset
943
kono
parents:
diff changeset
944 /* Sanitize all ASAN_MARK unpoison calls that are not reachable by a BB
kono
parents:
diff changeset
945 that contains an ASAN_MARK poison. All these ASAN_MARK unpoison call
kono
parents:
diff changeset
946 can be removed as all variables are unpoisoned in a function prologue. */
kono
parents:
diff changeset
947
kono
parents:
diff changeset
948 static void
kono
parents:
diff changeset
949 sanitize_asan_mark_unpoison (void)
kono
parents:
diff changeset
950 {
kono
parents:
diff changeset
951 /* 1) Find all BBs that contain an ASAN_MARK poison call. */
kono
parents:
diff changeset
952 auto_sbitmap with_poison (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
953 bitmap_clear (with_poison);
kono
parents:
diff changeset
954 basic_block bb;
kono
parents:
diff changeset
955
kono
parents:
diff changeset
956 FOR_EACH_BB_FN (bb, cfun)
kono
parents:
diff changeset
957 {
kono
parents:
diff changeset
958 if (bitmap_bit_p (with_poison, bb->index))
kono
parents:
diff changeset
959 continue;
kono
parents:
diff changeset
960
kono
parents:
diff changeset
961 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
962 for (gsi = gsi_last_bb (bb); !gsi_end_p (gsi); gsi_prev (&gsi))
kono
parents:
diff changeset
963 {
kono
parents:
diff changeset
964 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
965 if (asan_mark_p (stmt, ASAN_MARK_POISON))
kono
parents:
diff changeset
966 {
kono
parents:
diff changeset
967 bitmap_set_bit (with_poison, bb->index);
kono
parents:
diff changeset
968 break;
kono
parents:
diff changeset
969 }
kono
parents:
diff changeset
970 }
kono
parents:
diff changeset
971 }
kono
parents:
diff changeset
972
kono
parents:
diff changeset
973 auto_sbitmap poisoned (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
974 bitmap_clear (poisoned);
kono
parents:
diff changeset
975 auto_sbitmap worklist (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
976 bitmap_copy (worklist, with_poison);
kono
parents:
diff changeset
977
kono
parents:
diff changeset
978 /* 2) Propagate the information to all reachable blocks. */
kono
parents:
diff changeset
979 while (!bitmap_empty_p (worklist))
kono
parents:
diff changeset
980 {
kono
parents:
diff changeset
981 unsigned i = bitmap_first_set_bit (worklist);
kono
parents:
diff changeset
982 bitmap_clear_bit (worklist, i);
kono
parents:
diff changeset
983 basic_block bb = BASIC_BLOCK_FOR_FN (cfun, i);
kono
parents:
diff changeset
984 gcc_assert (bb);
kono
parents:
diff changeset
985
kono
parents:
diff changeset
986 edge e;
kono
parents:
diff changeset
987 edge_iterator ei;
kono
parents:
diff changeset
988 FOR_EACH_EDGE (e, ei, bb->succs)
kono
parents:
diff changeset
989 if (!bitmap_bit_p (poisoned, e->dest->index))
kono
parents:
diff changeset
990 {
kono
parents:
diff changeset
991 bitmap_set_bit (poisoned, e->dest->index);
kono
parents:
diff changeset
992 bitmap_set_bit (worklist, e->dest->index);
kono
parents:
diff changeset
993 }
kono
parents:
diff changeset
994 }
kono
parents:
diff changeset
995
kono
parents:
diff changeset
996 /* 3) Iterate all BBs not included in POISONED BBs and remove unpoison
kono
parents:
diff changeset
997 ASAN_MARK preceding an ASAN_MARK poison (which can still happen). */
kono
parents:
diff changeset
998 FOR_EACH_BB_FN (bb, cfun)
kono
parents:
diff changeset
999 {
kono
parents:
diff changeset
1000 if (bitmap_bit_p (poisoned, bb->index))
kono
parents:
diff changeset
1001 continue;
kono
parents:
diff changeset
1002
kono
parents:
diff changeset
1003 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1004 for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi);)
kono
parents:
diff changeset
1005 {
kono
parents:
diff changeset
1006 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1007 if (gimple_call_internal_p (stmt, IFN_ASAN_MARK))
kono
parents:
diff changeset
1008 {
kono
parents:
diff changeset
1009 if (asan_mark_p (stmt, ASAN_MARK_POISON))
kono
parents:
diff changeset
1010 break;
kono
parents:
diff changeset
1011 else
kono
parents:
diff changeset
1012 {
kono
parents:
diff changeset
1013 if (dump_file)
kono
parents:
diff changeset
1014 fprintf (dump_file, "Removing ASAN_MARK unpoison\n");
kono
parents:
diff changeset
1015 unlink_stmt_vdef (stmt);
kono
parents:
diff changeset
1016 release_defs (stmt);
kono
parents:
diff changeset
1017 gsi_remove (&gsi, true);
kono
parents:
diff changeset
1018 continue;
kono
parents:
diff changeset
1019 }
kono
parents:
diff changeset
1020 }
kono
parents:
diff changeset
1021
kono
parents:
diff changeset
1022 gsi_next (&gsi);
kono
parents:
diff changeset
1023 }
kono
parents:
diff changeset
1024 }
kono
parents:
diff changeset
1025 }
kono
parents:
diff changeset
1026
kono
parents:
diff changeset
1027 /* Return true when STMT is either ASAN_CHECK call or a call of a function
kono
parents:
diff changeset
1028 that can contain an ASAN_CHECK. */
kono
parents:
diff changeset
1029
kono
parents:
diff changeset
1030 static bool
kono
parents:
diff changeset
1031 maybe_contains_asan_check (gimple *stmt)
kono
parents:
diff changeset
1032 {
kono
parents:
diff changeset
1033 if (is_gimple_call (stmt))
kono
parents:
diff changeset
1034 {
kono
parents:
diff changeset
1035 if (gimple_call_internal_p (stmt, IFN_ASAN_MARK))
kono
parents:
diff changeset
1036 return false;
kono
parents:
diff changeset
1037 else
kono
parents:
diff changeset
1038 return !(gimple_call_flags (stmt) & ECF_CONST);
kono
parents:
diff changeset
1039 }
kono
parents:
diff changeset
1040 else if (is_a<gasm *> (stmt))
kono
parents:
diff changeset
1041 return true;
kono
parents:
diff changeset
1042
kono
parents:
diff changeset
1043 return false;
kono
parents:
diff changeset
1044 }
kono
parents:
diff changeset
1045
kono
parents:
diff changeset
1046 /* Sanitize all ASAN_MARK poison calls that are not followed by an ASAN_CHECK
kono
parents:
diff changeset
1047 call. These calls can be removed. */
kono
parents:
diff changeset
1048
kono
parents:
diff changeset
1049 static void
kono
parents:
diff changeset
1050 sanitize_asan_mark_poison (void)
kono
parents:
diff changeset
1051 {
kono
parents:
diff changeset
1052 /* 1) Find all BBs that possibly contain an ASAN_CHECK. */
kono
parents:
diff changeset
1053 auto_sbitmap with_check (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
1054 bitmap_clear (with_check);
kono
parents:
diff changeset
1055 basic_block bb;
kono
parents:
diff changeset
1056
kono
parents:
diff changeset
1057 FOR_EACH_BB_FN (bb, cfun)
kono
parents:
diff changeset
1058 {
kono
parents:
diff changeset
1059 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1060 for (gsi = gsi_last_bb (bb); !gsi_end_p (gsi); gsi_prev (&gsi))
kono
parents:
diff changeset
1061 {
kono
parents:
diff changeset
1062 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1063 if (maybe_contains_asan_check (stmt))
kono
parents:
diff changeset
1064 {
kono
parents:
diff changeset
1065 bitmap_set_bit (with_check, bb->index);
kono
parents:
diff changeset
1066 break;
kono
parents:
diff changeset
1067 }
kono
parents:
diff changeset
1068 }
kono
parents:
diff changeset
1069 }
kono
parents:
diff changeset
1070
kono
parents:
diff changeset
1071 auto_sbitmap can_reach_check (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
1072 bitmap_clear (can_reach_check);
kono
parents:
diff changeset
1073 auto_sbitmap worklist (last_basic_block_for_fn (cfun) + 1);
kono
parents:
diff changeset
1074 bitmap_copy (worklist, with_check);
kono
parents:
diff changeset
1075
kono
parents:
diff changeset
1076 /* 2) Propagate the information to all definitions blocks. */
kono
parents:
diff changeset
1077 while (!bitmap_empty_p (worklist))
kono
parents:
diff changeset
1078 {
kono
parents:
diff changeset
1079 unsigned i = bitmap_first_set_bit (worklist);
kono
parents:
diff changeset
1080 bitmap_clear_bit (worklist, i);
kono
parents:
diff changeset
1081 basic_block bb = BASIC_BLOCK_FOR_FN (cfun, i);
kono
parents:
diff changeset
1082 gcc_assert (bb);
kono
parents:
diff changeset
1083
kono
parents:
diff changeset
1084 edge e;
kono
parents:
diff changeset
1085 edge_iterator ei;
kono
parents:
diff changeset
1086 FOR_EACH_EDGE (e, ei, bb->preds)
kono
parents:
diff changeset
1087 if (!bitmap_bit_p (can_reach_check, e->src->index))
kono
parents:
diff changeset
1088 {
kono
parents:
diff changeset
1089 bitmap_set_bit (can_reach_check, e->src->index);
kono
parents:
diff changeset
1090 bitmap_set_bit (worklist, e->src->index);
kono
parents:
diff changeset
1091 }
kono
parents:
diff changeset
1092 }
kono
parents:
diff changeset
1093
kono
parents:
diff changeset
1094 /* 3) Iterate all BBs not included in CAN_REACH_CHECK BBs and remove poison
kono
parents:
diff changeset
1095 ASAN_MARK not followed by a call to function having an ASAN_CHECK. */
kono
parents:
diff changeset
1096 FOR_EACH_BB_FN (bb, cfun)
kono
parents:
diff changeset
1097 {
kono
parents:
diff changeset
1098 if (bitmap_bit_p (can_reach_check, bb->index))
kono
parents:
diff changeset
1099 continue;
kono
parents:
diff changeset
1100
kono
parents:
diff changeset
1101 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1102 for (gsi = gsi_last_bb (bb); !gsi_end_p (gsi);)
kono
parents:
diff changeset
1103 {
kono
parents:
diff changeset
1104 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1105 if (maybe_contains_asan_check (stmt))
kono
parents:
diff changeset
1106 break;
kono
parents:
diff changeset
1107 else if (asan_mark_p (stmt, ASAN_MARK_POISON))
kono
parents:
diff changeset
1108 {
kono
parents:
diff changeset
1109 if (dump_file)
kono
parents:
diff changeset
1110 fprintf (dump_file, "Removing ASAN_MARK poison\n");
kono
parents:
diff changeset
1111 unlink_stmt_vdef (stmt);
kono
parents:
diff changeset
1112 release_defs (stmt);
kono
parents:
diff changeset
1113 gimple_stmt_iterator gsi2 = gsi;
kono
parents:
diff changeset
1114 gsi_prev (&gsi);
kono
parents:
diff changeset
1115 gsi_remove (&gsi2, true);
kono
parents:
diff changeset
1116 continue;
kono
parents:
diff changeset
1117 }
kono
parents:
diff changeset
1118
kono
parents:
diff changeset
1119 gsi_prev (&gsi);
kono
parents:
diff changeset
1120 }
kono
parents:
diff changeset
1121 }
kono
parents:
diff changeset
1122 }
kono
parents:
diff changeset
1123
kono
parents:
diff changeset
1124 /* Rewrite all usages of tree OP which is a PARM_DECL with a VAR_DECL
kono
parents:
diff changeset
1125 that is it's DECL_VALUE_EXPR. */
kono
parents:
diff changeset
1126
kono
parents:
diff changeset
1127 static tree
kono
parents:
diff changeset
1128 rewrite_usage_of_param (tree *op, int *walk_subtrees, void *)
kono
parents:
diff changeset
1129 {
kono
parents:
diff changeset
1130 if (TREE_CODE (*op) == PARM_DECL && DECL_HAS_VALUE_EXPR_P (*op))
kono
parents:
diff changeset
1131 {
kono
parents:
diff changeset
1132 *op = DECL_VALUE_EXPR (*op);
kono
parents:
diff changeset
1133 *walk_subtrees = 0;
kono
parents:
diff changeset
1134 }
kono
parents:
diff changeset
1135
kono
parents:
diff changeset
1136 return NULL;
kono
parents:
diff changeset
1137 }
kono
parents:
diff changeset
1138
kono
parents:
diff changeset
1139 /* For a given function FUN, rewrite all addressable parameters so that
kono
parents:
diff changeset
1140 a new automatic variable is introduced. Right after function entry
kono
parents:
diff changeset
1141 a parameter is assigned to the variable. */
kono
parents:
diff changeset
1142
kono
parents:
diff changeset
1143 static void
kono
parents:
diff changeset
1144 sanitize_rewrite_addressable_params (function *fun)
kono
parents:
diff changeset
1145 {
kono
parents:
diff changeset
1146 gimple *g;
kono
parents:
diff changeset
1147 gimple_seq stmts = NULL;
kono
parents:
diff changeset
1148 bool has_any_addressable_param = false;
kono
parents:
diff changeset
1149 auto_vec<tree> clear_value_expr_list;
kono
parents:
diff changeset
1150
kono
parents:
diff changeset
1151 for (tree arg = DECL_ARGUMENTS (current_function_decl);
kono
parents:
diff changeset
1152 arg; arg = DECL_CHAIN (arg))
kono
parents:
diff changeset
1153 {
kono
parents:
diff changeset
1154 tree type = TREE_TYPE (arg);
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1155 if (TREE_ADDRESSABLE (arg)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1156 && !TREE_ADDRESSABLE (type)
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1157 && !TREE_THIS_VOLATILE (arg)
111
kono
parents:
diff changeset
1158 && TREE_CODE (TYPE_SIZE (type)) == INTEGER_CST)
kono
parents:
diff changeset
1159 {
kono
parents:
diff changeset
1160 TREE_ADDRESSABLE (arg) = 0;
kono
parents:
diff changeset
1161 /* The parameter is no longer addressable. */
kono
parents:
diff changeset
1162 has_any_addressable_param = true;
kono
parents:
diff changeset
1163
kono
parents:
diff changeset
1164 /* Create a new automatic variable. */
kono
parents:
diff changeset
1165 tree var = build_decl (DECL_SOURCE_LOCATION (arg),
kono
parents:
diff changeset
1166 VAR_DECL, DECL_NAME (arg), type);
kono
parents:
diff changeset
1167 TREE_ADDRESSABLE (var) = 1;
kono
parents:
diff changeset
1168 DECL_IGNORED_P (var) = 1;
kono
parents:
diff changeset
1169
kono
parents:
diff changeset
1170 gimple_add_tmp_var (var);
kono
parents:
diff changeset
1171
131
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1172 /* We skip parameters that have a DECL_VALUE_EXPR. */
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1173 if (DECL_HAS_VALUE_EXPR_P (arg))
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1174 continue;
84e7813d76e9 gcc-8.2
mir3636
parents: 111
diff changeset
1175
111
kono
parents:
diff changeset
1176 if (dump_file)
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1177 {
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1178 fprintf (dump_file,
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1179 "Rewriting parameter whose address is taken: ");
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1180 print_generic_expr (dump_file, arg, dump_flags);
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1181 fputc ('\n', dump_file);
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1182 }
111
kono
parents:
diff changeset
1183
kono
parents:
diff changeset
1184 SET_DECL_PT_UID (var, DECL_PT_UID (arg));
kono
parents:
diff changeset
1185
kono
parents:
diff changeset
1186 /* Assign value of parameter to newly created variable. */
kono
parents:
diff changeset
1187 if ((TREE_CODE (type) == COMPLEX_TYPE
kono
parents:
diff changeset
1188 || TREE_CODE (type) == VECTOR_TYPE))
kono
parents:
diff changeset
1189 {
kono
parents:
diff changeset
1190 /* We need to create a SSA name that will be used for the
kono
parents:
diff changeset
1191 assignment. */
kono
parents:
diff changeset
1192 DECL_GIMPLE_REG_P (arg) = 1;
kono
parents:
diff changeset
1193 tree tmp = get_or_create_ssa_default_def (cfun, arg);
kono
parents:
diff changeset
1194 g = gimple_build_assign (var, tmp);
kono
parents:
diff changeset
1195 gimple_set_location (g, DECL_SOURCE_LOCATION (arg));
kono
parents:
diff changeset
1196 gimple_seq_add_stmt (&stmts, g);
kono
parents:
diff changeset
1197 }
kono
parents:
diff changeset
1198 else
kono
parents:
diff changeset
1199 {
kono
parents:
diff changeset
1200 g = gimple_build_assign (var, arg);
kono
parents:
diff changeset
1201 gimple_set_location (g, DECL_SOURCE_LOCATION (arg));
kono
parents:
diff changeset
1202 gimple_seq_add_stmt (&stmts, g);
kono
parents:
diff changeset
1203 }
kono
parents:
diff changeset
1204
kono
parents:
diff changeset
1205 if (target_for_debug_bind (arg))
kono
parents:
diff changeset
1206 {
kono
parents:
diff changeset
1207 g = gimple_build_debug_bind (arg, var, NULL);
kono
parents:
diff changeset
1208 gimple_seq_add_stmt (&stmts, g);
kono
parents:
diff changeset
1209 clear_value_expr_list.safe_push (arg);
kono
parents:
diff changeset
1210 }
kono
parents:
diff changeset
1211
kono
parents:
diff changeset
1212 DECL_HAS_VALUE_EXPR_P (arg) = 1;
kono
parents:
diff changeset
1213 SET_DECL_VALUE_EXPR (arg, var);
kono
parents:
diff changeset
1214 }
kono
parents:
diff changeset
1215 }
kono
parents:
diff changeset
1216
kono
parents:
diff changeset
1217 if (!has_any_addressable_param)
kono
parents:
diff changeset
1218 return;
kono
parents:
diff changeset
1219
kono
parents:
diff changeset
1220 /* Replace all usages of PARM_DECLs with the newly
kono
parents:
diff changeset
1221 created variable VAR. */
kono
parents:
diff changeset
1222 basic_block bb;
kono
parents:
diff changeset
1223 FOR_EACH_BB_FN (bb, fun)
kono
parents:
diff changeset
1224 {
kono
parents:
diff changeset
1225 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1226 for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
kono
parents:
diff changeset
1227 {
kono
parents:
diff changeset
1228 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1229 gimple_stmt_iterator it = gsi_for_stmt (stmt);
kono
parents:
diff changeset
1230 walk_gimple_stmt (&it, NULL, rewrite_usage_of_param, NULL);
kono
parents:
diff changeset
1231 }
kono
parents:
diff changeset
1232 for (gsi = gsi_start_phis (bb); !gsi_end_p (gsi); gsi_next (&gsi))
kono
parents:
diff changeset
1233 {
kono
parents:
diff changeset
1234 gphi *phi = dyn_cast<gphi *> (gsi_stmt (gsi));
kono
parents:
diff changeset
1235 for (unsigned i = 0; i < gimple_phi_num_args (phi); ++i)
kono
parents:
diff changeset
1236 {
kono
parents:
diff changeset
1237 hash_set<tree> visited_nodes;
kono
parents:
diff changeset
1238 walk_tree (gimple_phi_arg_def_ptr (phi, i),
kono
parents:
diff changeset
1239 rewrite_usage_of_param, NULL, &visited_nodes);
kono
parents:
diff changeset
1240 }
kono
parents:
diff changeset
1241 }
kono
parents:
diff changeset
1242 }
kono
parents:
diff changeset
1243
kono
parents:
diff changeset
1244 /* Unset value expr for parameters for which we created debug bind
kono
parents:
diff changeset
1245 expressions. */
kono
parents:
diff changeset
1246 unsigned i;
kono
parents:
diff changeset
1247 tree arg;
kono
parents:
diff changeset
1248 FOR_EACH_VEC_ELT (clear_value_expr_list, i, arg)
kono
parents:
diff changeset
1249 {
kono
parents:
diff changeset
1250 DECL_HAS_VALUE_EXPR_P (arg) = 0;
kono
parents:
diff changeset
1251 SET_DECL_VALUE_EXPR (arg, NULL_TREE);
kono
parents:
diff changeset
1252 }
kono
parents:
diff changeset
1253
kono
parents:
diff changeset
1254 /* Insert default assignments at the beginning of a function. */
kono
parents:
diff changeset
1255 basic_block entry_bb = ENTRY_BLOCK_PTR_FOR_FN (fun);
kono
parents:
diff changeset
1256 entry_bb = split_edge (single_succ_edge (entry_bb));
kono
parents:
diff changeset
1257
kono
parents:
diff changeset
1258 gimple_stmt_iterator gsi = gsi_start_bb (entry_bb);
kono
parents:
diff changeset
1259 gsi_insert_seq_before (&gsi, stmts, GSI_NEW_STMT);
kono
parents:
diff changeset
1260 }
kono
parents:
diff changeset
1261
kono
parents:
diff changeset
1262 unsigned int
kono
parents:
diff changeset
1263 pass_sanopt::execute (function *fun)
kono
parents:
diff changeset
1264 {
kono
parents:
diff changeset
1265 basic_block bb;
kono
parents:
diff changeset
1266 int asan_num_accesses = 0;
kono
parents:
diff changeset
1267 bool contains_asan_mark = false;
kono
parents:
diff changeset
1268
kono
parents:
diff changeset
1269 /* Try to remove redundant checks. */
kono
parents:
diff changeset
1270 if (optimize
kono
parents:
diff changeset
1271 && (flag_sanitize
kono
parents:
diff changeset
1272 & (SANITIZE_NULL | SANITIZE_ALIGNMENT
kono
parents:
diff changeset
1273 | SANITIZE_ADDRESS | SANITIZE_VPTR | SANITIZE_POINTER_OVERFLOW)))
kono
parents:
diff changeset
1274 asan_num_accesses = sanopt_optimize (fun, &contains_asan_mark);
kono
parents:
diff changeset
1275 else if (flag_sanitize & SANITIZE_ADDRESS)
kono
parents:
diff changeset
1276 {
kono
parents:
diff changeset
1277 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1278 FOR_EACH_BB_FN (bb, fun)
kono
parents:
diff changeset
1279 for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
kono
parents:
diff changeset
1280 {
kono
parents:
diff changeset
1281 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1282 if (gimple_call_internal_p (stmt, IFN_ASAN_CHECK))
kono
parents:
diff changeset
1283 ++asan_num_accesses;
kono
parents:
diff changeset
1284 else if (gimple_call_internal_p (stmt, IFN_ASAN_MARK))
kono
parents:
diff changeset
1285 contains_asan_mark = true;
kono
parents:
diff changeset
1286 }
kono
parents:
diff changeset
1287 }
kono
parents:
diff changeset
1288
kono
parents:
diff changeset
1289 if (contains_asan_mark)
kono
parents:
diff changeset
1290 {
kono
parents:
diff changeset
1291 sanitize_asan_mark_unpoison ();
kono
parents:
diff changeset
1292 sanitize_asan_mark_poison ();
kono
parents:
diff changeset
1293 }
kono
parents:
diff changeset
1294
kono
parents:
diff changeset
1295 if (asan_sanitize_stack_p ())
kono
parents:
diff changeset
1296 sanitize_rewrite_addressable_params (fun);
kono
parents:
diff changeset
1297
145
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1298 bool use_calls = param_asan_instrumentation_with_call_threshold < INT_MAX
1830386684a0 gcc-9.2.0
anatofuz
parents: 131
diff changeset
1299 && asan_num_accesses >= param_asan_instrumentation_with_call_threshold;
111
kono
parents:
diff changeset
1300
kono
parents:
diff changeset
1301 hash_map<tree, tree> shadow_vars_mapping;
kono
parents:
diff changeset
1302 bool need_commit_edge_insert = false;
kono
parents:
diff changeset
1303 FOR_EACH_BB_FN (bb, fun)
kono
parents:
diff changeset
1304 {
kono
parents:
diff changeset
1305 gimple_stmt_iterator gsi;
kono
parents:
diff changeset
1306 for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); )
kono
parents:
diff changeset
1307 {
kono
parents:
diff changeset
1308 gimple *stmt = gsi_stmt (gsi);
kono
parents:
diff changeset
1309 bool no_next = false;
kono
parents:
diff changeset
1310
kono
parents:
diff changeset
1311 if (!is_gimple_call (stmt))
kono
parents:
diff changeset
1312 {
kono
parents:
diff changeset
1313 gsi_next (&gsi);
kono
parents:
diff changeset
1314 continue;
kono
parents:
diff changeset
1315 }
kono
parents:
diff changeset
1316
kono
parents:
diff changeset
1317 if (gimple_call_internal_p (stmt))
kono
parents:
diff changeset
1318 {
kono
parents:
diff changeset
1319 enum internal_fn ifn = gimple_call_internal_fn (stmt);
kono
parents:
diff changeset
1320 switch (ifn)
kono
parents:
diff changeset
1321 {
kono
parents:
diff changeset
1322 case IFN_UBSAN_NULL:
kono
parents:
diff changeset
1323 no_next = ubsan_expand_null_ifn (&gsi);
kono
parents:
diff changeset
1324 break;
kono
parents:
diff changeset
1325 case IFN_UBSAN_BOUNDS:
kono
parents:
diff changeset
1326 no_next = ubsan_expand_bounds_ifn (&gsi);
kono
parents:
diff changeset
1327 break;
kono
parents:
diff changeset
1328 case IFN_UBSAN_OBJECT_SIZE:
kono
parents:
diff changeset
1329 no_next = ubsan_expand_objsize_ifn (&gsi);
kono
parents:
diff changeset
1330 break;
kono
parents:
diff changeset
1331 case IFN_UBSAN_PTR:
kono
parents:
diff changeset
1332 no_next = ubsan_expand_ptr_ifn (&gsi);
kono
parents:
diff changeset
1333 break;
kono
parents:
diff changeset
1334 case IFN_UBSAN_VPTR:
kono
parents:
diff changeset
1335 no_next = ubsan_expand_vptr_ifn (&gsi);
kono
parents:
diff changeset
1336 break;
kono
parents:
diff changeset
1337 case IFN_ASAN_CHECK:
kono
parents:
diff changeset
1338 no_next = asan_expand_check_ifn (&gsi, use_calls);
kono
parents:
diff changeset
1339 break;
kono
parents:
diff changeset
1340 case IFN_ASAN_MARK:
kono
parents:
diff changeset
1341 no_next = asan_expand_mark_ifn (&gsi);
kono
parents:
diff changeset
1342 break;
kono
parents:
diff changeset
1343 case IFN_ASAN_POISON:
kono
parents:
diff changeset
1344 no_next = asan_expand_poison_ifn (&gsi,
kono
parents:
diff changeset
1345 &need_commit_edge_insert,
kono
parents:
diff changeset
1346 shadow_vars_mapping);
kono
parents:
diff changeset
1347 break;
kono
parents:
diff changeset
1348 default:
kono
parents:
diff changeset
1349 break;
kono
parents:
diff changeset
1350 }
kono
parents:
diff changeset
1351 }
kono
parents:
diff changeset
1352 else if (gimple_call_builtin_p (stmt, BUILT_IN_NORMAL))
kono
parents:
diff changeset
1353 {
kono
parents:
diff changeset
1354 tree callee = gimple_call_fndecl (stmt);
kono
parents:
diff changeset
1355 switch (DECL_FUNCTION_CODE (callee))
kono
parents:
diff changeset
1356 {
kono
parents:
diff changeset
1357 case BUILT_IN_UNREACHABLE:
kono
parents:
diff changeset
1358 if (sanitize_flags_p (SANITIZE_UNREACHABLE))
kono
parents:
diff changeset
1359 no_next = ubsan_instrument_unreachable (&gsi);
kono
parents:
diff changeset
1360 break;
kono
parents:
diff changeset
1361 default:
kono
parents:
diff changeset
1362 break;
kono
parents:
diff changeset
1363 }
kono
parents:
diff changeset
1364 }
kono
parents:
diff changeset
1365
kono
parents:
diff changeset
1366 if (dump_file && (dump_flags & TDF_DETAILS))
kono
parents:
diff changeset
1367 {
kono
parents:
diff changeset
1368 fprintf (dump_file, "Expanded: ");
kono
parents:
diff changeset
1369 print_gimple_stmt (dump_file, stmt, 0, dump_flags);
kono
parents:
diff changeset
1370 }
kono
parents:
diff changeset
1371
kono
parents:
diff changeset
1372 if (!no_next)
kono
parents:
diff changeset
1373 gsi_next (&gsi);
kono
parents:
diff changeset
1374 }
kono
parents:
diff changeset
1375 }
kono
parents:
diff changeset
1376
kono
parents:
diff changeset
1377 if (need_commit_edge_insert)
kono
parents:
diff changeset
1378 gsi_commit_edge_inserts ();
kono
parents:
diff changeset
1379
kono
parents:
diff changeset
1380 return 0;
kono
parents:
diff changeset
1381 }
kono
parents:
diff changeset
1382
kono
parents:
diff changeset
1383 } // anon namespace
kono
parents:
diff changeset
1384
kono
parents:
diff changeset
1385 gimple_opt_pass *
kono
parents:
diff changeset
1386 make_pass_sanopt (gcc::context *ctxt)
kono
parents:
diff changeset
1387 {
kono
parents:
diff changeset
1388 return new pass_sanopt (ctxt);
kono
parents:
diff changeset
1389 }