annotate gcc/sanopt.c @ 143:76e1cf5455ef

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