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);
     }

Reply via email to