Hi, This patch removes redundant gimple_build_nop calls from tree-chkp.c. MPX-bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk.
Thanks, Ilya -- 2015-05-20 Ilya Enkovich <enkovich....@gmail.com> * tree-chkp.c (chkp_maybe_copy_and_register_bounds): Remove useless gimple_build_nop calls. (chkp_find_bounds_for_elem): Likewise. (chkp_get_zero_bounds): Likewise. (chkp_get_none_bounds): Likewise. (chkp_get_bounds_by_definition): Likewise. (chkp_generate_extern_var_bounds): Likewise. (chkp_get_bounds_for_decl_addr): Likewise. (chkp_get_bounds_for_string_cst): Likewise. diff --git a/gcc/tree-chkp.c b/gcc/tree-chkp.c index 288470b..4f84a22 100644 --- a/gcc/tree-chkp.c +++ b/gcc/tree-chkp.c @@ -1172,10 +1172,10 @@ chkp_maybe_copy_and_register_bounds (tree ptr, tree bnd) gimple_stmt_iterator gsi; if (bnd_var) - copy = make_ssa_name (bnd_var, gimple_build_nop ()); + copy = make_ssa_name (bnd_var); else copy = make_temp_ssa_name (pointer_bounds_type_node, - gimple_build_nop (), + NULL, CHKP_BOUND_TMP_NAME); assign = gimple_build_assign (copy, bnd); @@ -1534,7 +1534,7 @@ chkp_find_bounds_for_elem (tree elem, tree *all_bounds, { if (!all_bounds[offs / POINTER_SIZE]) { - tree temp = make_temp_ssa_name (type, gimple_build_nop (), ""); + tree temp = make_temp_ssa_name (type, NULL, ""); gimple assign = gimple_build_assign (temp, elem); gimple_stmt_iterator gsi; @@ -2043,7 +2043,7 @@ chkp_get_zero_bounds (void) gimple_stmt_iterator gsi = gsi_start_bb (chkp_get_entry_block ()); gimple stmt; - zero_bounds = chkp_get_tmp_reg (gimple_build_nop ()); + zero_bounds = chkp_get_tmp_reg (NULL); stmt = gimple_build_assign (zero_bounds, chkp_get_zero_bounds_var ()); gsi_insert_before (&gsi, stmt, GSI_SAME_STMT); } @@ -2073,7 +2073,7 @@ chkp_get_none_bounds (void) gimple_stmt_iterator gsi = gsi_start_bb (chkp_get_entry_block ()); gimple stmt; - none_bounds = chkp_get_tmp_reg (gimple_build_nop ()); + none_bounds = chkp_get_tmp_reg (NULL); stmt = gimple_build_assign (none_bounds, chkp_get_none_bounds_var ()); gsi_insert_before (&gsi, stmt, GSI_SAME_STMT); } @@ -2728,7 +2728,7 @@ chkp_get_bounds_by_definition (tree node, gimple def_stmt, var = chkp_get_bounds_var (SSA_NAME_VAR (node)); else var = make_temp_ssa_name (pointer_bounds_type_node, - gimple_build_nop (), + NULL, CHKP_BOUND_TMP_NAME); else var = chkp_get_tmp_var (); @@ -2908,7 +2908,7 @@ chkp_generate_extern_var_bounds (tree var) gimple_seq_add_stmt (&seq, stmt); lb = chkp_build_addr_expr (var); - size = make_ssa_name (chkp_get_size_tmp_var (), gimple_build_nop ()); + size = make_ssa_name (chkp_get_size_tmp_var ()); if (flag_chkp_zero_dynamic_size_as_infinite) { @@ -3005,7 +3005,7 @@ chkp_get_bounds_for_decl_addr (tree decl) gimple_stmt_iterator gsi = gsi_start_bb (chkp_get_entry_block ()); gimple stmt; - bounds = chkp_get_tmp_reg (gimple_build_nop ()); + bounds = chkp_get_tmp_reg (NULL); stmt = gimple_build_assign (bounds, bnd_var); gsi_insert_before (&gsi, stmt, GSI_SAME_STMT); } @@ -3049,7 +3049,7 @@ chkp_get_bounds_for_string_cst (tree cst) gimple_stmt_iterator gsi = gsi_start_bb (chkp_get_entry_block ()); gimple stmt; - bounds = chkp_get_tmp_reg (gimple_build_nop ()); + bounds = chkp_get_tmp_reg (NULL); stmt = gimple_build_assign (bounds, bnd_var); gsi_insert_before (&gsi, stmt, GSI_SAME_STMT); }