[PATCH] gimple_{build_assign,assign_set_rhs}_with_ops* cleanup
Richard Biener
rguenther@suse.de
Wed Nov 19 14:54:00 GMT 2014
On Wed, 19 Nov 2014, Jakub Jelinek wrote:
> Hi!
>
> This patch:
> 1) adds unary op overload to gimple_build_assign_with_ops
> so that many callers don't need to pass NULL_TREE as the last
> argument explicitly
> 2) adds unary op overload to gimple_assign_set_rhs_with_ops for similar
> reasons
> 3) renames gimple_assign_set_rhs_with_ops_1 to
> gimple_assign_set_rhs_with_ops so it becomes ternary op overload
> to the existing binary op one and 2) added unary op
> and adjusts all users that I found.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
Ok.
Thanks,
Richard.
> 2014-11-19 Jakub Jelinek <jakub@redhat.com>
>
> * gimple.h (gimple_build_assign_with_ops): Add unary arg overload.
> (gimple_assign_set_rhs_with_ops_1): Renamed to ...
> (gimple_assign_set_rhs_with_ops): ... this. Adjust binary arg
> inline overload to use it. Add unary arg overload.
> * gimple.c (gimple_build_assign_with_ops): New unary arg overload.
> (gimple_assign_set_rhs_from_tree): Use
> gimple_assign_set_rhs_with_ops instead of
> gimple_assign_set_rhs_with_ops_1.
> (gimple_assign_set_rhs_with_ops_1): Renamed to ...
> (gimple_assign_set_rhs_with_ops): ... this.
> * ipa-split.c (split_function): Remove last NULL argument
> from gimple_build_assign_with_ops call.
> * tree-ssa-loop-im.c
> (move_computations_dom_walker::before_dom_children): Likewise.
> * tsan.c (instrument_builtin_call): Likewise.
> * tree-vect-stmts.c (vect_init_vector, vectorizable_mask_load_store,
> vectorizable_conversion, vectorizable_load): Likewise.
> * tree-vect-loop.c (vect_is_simple_reduction_1,
> get_initial_def_for_induction): Likewise.
> * tree-loop-distribution.c (generate_memset_builtin): Likewise.
> * tree-vect-patterns.c (vect_handle_widen_op_by_const,
> vect_recog_widen_mult_pattern, vect_operation_fits_smaller_type,
> vect_recog_over_widening_pattern, vect_recog_rotate_pattern,
> vect_recog_vector_vector_shift_pattern, vect_recog_divmod_pattern,
> vect_recog_mixed_size_cond_pattern, adjust_bool_pattern_cast,
> adjust_bool_pattern, vect_recog_bool_pattern): Likewise.
> * tree-ssa-phiopt.c (conditional_replacement, abs_replacement,
> neg_replacement): Likewise.
> * asan.c (build_shadow_mem_access, maybe_create_ssa_name,
> maybe_cast_to_ptrmode, asan_expand_check_ifn): Likewise.
> * tree-vect-slp.c (vect_get_constant_vectors): Likewise.
> * omp-low.c (lower_rec_input_clauses, expand_omp_for_generic,
> expand_omp_for_static_nochunk, expand_omp_for_static_chunk,
> simd_clone_adjust): Likewise.
> * tree-vect-loop-manip.c (vect_create_cond_for_align_checks): Likewise.
> * gimple-ssa-strength-reduction.c (introduce_cast_before_cand,
> replace_one_candidate): Likewise.
> * gimple-builder.c (build_type_cast): Likewise.
> * tree-ssa-forwprop.c (simplify_rotate): Likewise.
> (forward_propagate_addr_expr_1): Remove last NULL argument
> from gimple_assign_set_rhs_with_ops call.
> (simplify_vector_constructor): Use gimple_assign_set_rhs_with_ops
> instead of gimple_assign_set_rhs_with_ops_1.
> * tree-ssa-reassoc.c (maybe_optimize_range_tests): Remove last NULL
> argument from gimple_build_assign_with_ops call.
> (repropagate_negates): Remove last NULL argument from
> gimple_assign_set_rhs_with_ops call.
> * ubsan.c (ubsan_expand_null_ifn, ubsan_expand_objsize_ifn): Remove
> last NULL argument from gimple_build_assign_with_ops call.
> (instrument_bool_enum_load): Likewise. Remove last NULL argument
> from gimple_assign_set_rhs_with_ops call.
> * tree-ssa-math-opts.c (build_and_insert_cast, convert_mult_to_fma):
> Remove last NULL argument from gimple_build_assign_with_ops call.
> (bswap_replace): Likewise. Use gimple_assign_set_rhs_with_ops instead
> of gimple_assign_set_rhs_with_ops_1.
> (convert_plusminus_to_widen): Use gimple_assign_set_rhs_with_ops
> instead of gimple_assign_set_rhs_with_ops_1.
> * gimple-fold.c (replace_stmt_with_simplification): Likewise.
> (rewrite_to_defined_overflow, gimple_build): Remove last NULL argument
> from gimple_build_assign_with_ops call.
> * tree-ssa-strlen.c (handle_pointer_plus): Remove last NULL argument
> from gimple_assign_set_rhs_with_ops call.
> * tree-vrp.c (simplify_truth_ops_using_ranges,
> simplify_bit_ops_using_ranges): Remove last NULL argument from
> gimple_assign_set_rhs_with_ops call.
> (simplify_float_conversion_using_ranges,
> simplify_internal_call_using_ranges): Remove last NULL argument from
> gimple_build_assign_with_ops call.
>
> --- gcc/gimple.h.jj 2014-11-19 10:45:17.780769013 +0100
> +++ gcc/gimple.h 2014-11-19 10:52:52.118574835 +0100
> @@ -1180,6 +1180,8 @@ gimple gimple_build_assign_with_ops (enu
> tree, tree, tree CXX_MEM_STAT_INFO);
> gimple gimple_build_assign_with_ops (enum tree_code, tree,
> tree, tree CXX_MEM_STAT_INFO);
> +gimple gimple_build_assign_with_ops (enum tree_code, tree,
> + tree CXX_MEM_STAT_INFO);
> gimple gimple_build_cond (enum tree_code, tree, tree, tree, tree);
> gimple gimple_build_cond_from_tree (tree, tree, tree);
> void gimple_cond_set_condition_from_tree (gimple, tree);
> @@ -1244,8 +1246,8 @@ bool gimple_assign_ssa_name_copy_p (gimp
> bool gimple_assign_unary_nop_p (gimple);
> void gimple_set_bb (gimple, basic_block);
> void gimple_assign_set_rhs_from_tree (gimple_stmt_iterator *, tree);
> -void gimple_assign_set_rhs_with_ops_1 (gimple_stmt_iterator *, enum tree_code,
> - tree, tree, tree);
> +void gimple_assign_set_rhs_with_ops (gimple_stmt_iterator *, enum tree_code,
> + tree, tree, tree);
> tree gimple_get_lhs (const_gimple);
> void gimple_set_lhs (gimple, tree);
> gimple gimple_copy (gimple);
> @@ -2313,14 +2315,24 @@ gimple_assign_set_rhs3 (gimple gs, tree
> gimple_set_op (gs, 3, rhs);
> }
>
> -/* A wrapper around gimple_assign_set_rhs_with_ops_1, for callers which expect
> - to see only a maximum of two operands. */
> +/* A wrapper around 3 operand gimple_assign_set_rhs_with_ops, for callers
> + which expect to see only two operands. */
>
> static inline void
> gimple_assign_set_rhs_with_ops (gimple_stmt_iterator *gsi, enum tree_code code,
> tree op1, tree op2)
> {
> - gimple_assign_set_rhs_with_ops_1 (gsi, code, op1, op2, NULL);
> + gimple_assign_set_rhs_with_ops (gsi, code, op1, op2, NULL);
> +}
> +
> +/* A wrapper around 3 operand gimple_assign_set_rhs_with_ops, for callers
> + which expect to see only one operands. */
> +
> +static inline void
> +gimple_assign_set_rhs_with_ops (gimple_stmt_iterator *gsi, enum tree_code code,
> + tree op1)
> +{
> + gimple_assign_set_rhs_with_ops (gsi, code, op1, NULL, NULL);
> }
>
> /* Returns true if GS is a nontemporal move. */
> --- gcc/gimple.c.jj 2014-11-19 10:45:17.763769320 +0100
> +++ gcc/gimple.c 2014-11-19 10:52:52.119574816 +0100
> @@ -446,6 +446,14 @@ gimple_build_assign_with_ops (enum tree_
> PASS_MEM_STAT);
> }
>
> +gimple
> +gimple_build_assign_with_ops (enum tree_code subcode, tree lhs, tree op1
> + MEM_STAT_DECL)
> +{
> + return gimple_build_assign_with_ops (subcode, lhs, op1, NULL_TREE, NULL_TREE
> + PASS_MEM_STAT);
> +}
> +
>
> /* Build a GIMPLE_COND statement.
>
> @@ -1550,7 +1558,7 @@ gimple_assign_set_rhs_from_tree (gimple_
> tree op1, op2, op3;
>
> extract_ops_from_tree_1 (expr, &subcode, &op1, &op2, &op3);
> - gimple_assign_set_rhs_with_ops_1 (gsi, subcode, op1, op2, op3);
> + gimple_assign_set_rhs_with_ops (gsi, subcode, op1, op2, op3);
> }
>
>
> @@ -1561,8 +1569,8 @@ gimple_assign_set_rhs_from_tree (gimple_
> did not have enough operand slots. */
>
> void
> -gimple_assign_set_rhs_with_ops_1 (gimple_stmt_iterator *gsi, enum tree_code code,
> - tree op1, tree op2, tree op3)
> +gimple_assign_set_rhs_with_ops (gimple_stmt_iterator *gsi, enum tree_code code,
> + tree op1, tree op2, tree op3)
> {
> unsigned new_rhs_ops = get_gimple_rhs_num_ops (code);
> gimple stmt = gsi_stmt (*gsi);
> --- gcc/ipa-split.c.jj 2014-11-11 00:06:19.000000000 +0100
> +++ gcc/ipa-split.c 2014-11-19 11:03:41.326884348 +0100
> @@ -1577,7 +1577,7 @@ split_function (struct split_point *spli
> tree tem = create_tmp_reg (restype, NULL);
> tem = make_ssa_name (tem, call);
> cpy = gimple_build_assign_with_ops (NOP_EXPR, retval,
> - tem, NULL_TREE);
> + tem);
> gsi_insert_after (&gsi, cpy, GSI_NEW_STMT);
> retval = tem;
> }
> --- gcc/tree-ssa-loop-im.c.jj 2014-10-28 14:39:56.000000000 +0100
> +++ gcc/tree-ssa-loop-im.c 2014-11-19 11:07:44.142513141 +0100
> @@ -1211,7 +1211,7 @@ move_computations_dom_walker::before_dom
> tree arg = PHI_ARG_DEF (stmt, 0);
> new_stmt = gimple_build_assign_with_ops (TREE_CODE (arg),
> gimple_phi_result (stmt),
> - arg, NULL_TREE);
> + arg);
> }
> else
> {
> --- gcc/tree-ssa-forwprop.c.jj 2014-11-18 08:26:47.000000000 +0100
> +++ gcc/tree-ssa-forwprop.c 2014-11-19 11:07:01.423281870 +0100
> @@ -757,10 +757,9 @@ forward_propagate_addr_expr_1 (tree name
>
> if (useless_type_conversion_p (TREE_TYPE (lhs), TREE_TYPE (new_def_rhs)))
> gimple_assign_set_rhs_with_ops (use_stmt_gsi, TREE_CODE (new_def_rhs),
> - new_def_rhs, NULL_TREE);
> + new_def_rhs);
> else if (is_gimple_min_invariant (new_def_rhs))
> - gimple_assign_set_rhs_with_ops (use_stmt_gsi, NOP_EXPR,
> - new_def_rhs, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (use_stmt_gsi, NOP_EXPR, new_def_rhs);
> else
> return false;
> gcc_assert (gsi_stmt (*use_stmt_gsi) == use_stmt);
> @@ -1725,8 +1724,7 @@ simplify_rotate (gimple_stmt_iterator *g
> {
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (TREE_TYPE (def_arg2[0]),
> - NULL),
> - rotcnt, NULL_TREE);
> + NULL), rotcnt);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> rotcnt = gimple_assign_lhs (g);
> }
> @@ -1740,7 +1738,7 @@ simplify_rotate (gimple_stmt_iterator *g
> {
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> g = gimple_build_assign_with_ops (NOP_EXPR, gimple_assign_lhs (stmt),
> - lhs, NULL_TREE);
> + lhs);
> }
> gsi_replace (gsi, g, false);
> return true;
> @@ -2055,7 +2053,7 @@ simplify_vector_constructor (gimple_stmt
> for (i = 0; i < nelts; i++)
> mask_elts[i] = build_int_cst (TREE_TYPE (mask_type), sel[i]);
> op2 = build_vector (mask_type, mask_elts);
> - gimple_assign_set_rhs_with_ops_1 (gsi, VEC_PERM_EXPR, orig, orig, op2);
> + gimple_assign_set_rhs_with_ops (gsi, VEC_PERM_EXPR, orig, orig, op2);
> }
> update_stmt (gsi_stmt (*gsi));
> return true;
> --- gcc/tsan.c.jj 2014-10-30 14:42:26.000000000 +0100
> +++ gcc/tsan.c 2014-11-19 11:20:35.024641177 +0100
> @@ -501,8 +501,7 @@ instrument_builtin_call (gimple_stmt_ite
> TREE_TYPE (args[1])))
> {
> tree var = make_ssa_name (TREE_TYPE (lhs), NULL);
> - g = gimple_build_assign_with_ops (NOP_EXPR, var,
> - args[1], NULL_TREE);
> + g = gimple_build_assign_with_ops (NOP_EXPR, var, args[1]);
> gsi_insert_after (gsi, g, GSI_NEW_STMT);
> args[1] = var;
> }
> @@ -516,8 +515,7 @@ instrument_builtin_call (gimple_stmt_ite
> gimple_call_lhs (stmt),
> args[1]);
> gsi_insert_after (gsi, g, GSI_NEW_STMT);
> - g = gimple_build_assign_with_ops (BIT_NOT_EXPR, lhs, var,
> - NULL_TREE);
> + g = gimple_build_assign_with_ops (BIT_NOT_EXPR, lhs, var);
> }
> else
> g = gimple_build_assign_with_ops (tsan_atomic_table[i].code,
> @@ -560,7 +558,7 @@ instrument_builtin_call (gimple_stmt_ite
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (TREE_TYPE (t),
> NULL),
> - args[1], NULL_TREE);
> + args[1]);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> args[1] = gimple_assign_lhs (g);
> }
> --- gcc/tree-vect-stmts.c.jj 2014-11-19 10:50:31.000000000 +0100
> +++ gcc/tree-vect-stmts.c 2014-11-19 11:18:16.389135912 +0100
> @@ -1333,8 +1333,7 @@ vect_init_vector (gimple stmt, tree val,
> {
> new_temp = make_ssa_name (TREE_TYPE (type), NULL);
> init_stmt = gimple_build_assign_with_ops (NOP_EXPR,
> - new_temp, val,
> - NULL_TREE);
> + new_temp, val);
> vect_init_vector_1 (stmt, init_stmt, gsi);
> val = new_temp;
> }
> @@ -1970,8 +1969,7 @@ vectorizable_mask_load_store (gimple stm
> var = make_ssa_name (var, NULL);
> op = build1 (VIEW_CONVERT_EXPR, idxtype, op);
> new_stmt
> - = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var,
> - op, NULL_TREE);
> + = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op);
> vect_finish_stmt_generation (stmt, new_stmt, gsi);
> op = var;
> }
> @@ -2001,7 +1999,7 @@ vectorizable_mask_load_store (gimple stm
> mask_op = build1 (VIEW_CONVERT_EXPR, masktype, mask_op);
> new_stmt
> = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var,
> - mask_op, NULL_TREE);
> + mask_op);
> vect_finish_stmt_generation (stmt, new_stmt, gsi);
> mask_op = var;
> }
> @@ -2022,8 +2020,7 @@ vectorizable_mask_load_store (gimple stm
> var = make_ssa_name (vec_dest, NULL);
> op = build1 (VIEW_CONVERT_EXPR, vectype, op);
> new_stmt
> - = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op,
> - NULL_TREE);
> + = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op);
> }
> else
> {
> @@ -3790,7 +3787,7 @@ vectorizable_conversion (gimple stmt, gi
> {
> gcc_assert (TREE_CODE_LENGTH (code1) == unary_op);
> new_stmt = gimple_build_assign_with_ops (code1, vec_dest,
> - vop0, NULL);
> + vop0);
> new_temp = make_ssa_name (vec_dest, new_stmt);
> gimple_assign_set_lhs (new_stmt, new_temp);
> }
> @@ -3902,8 +3899,7 @@ vectorizable_conversion (gimple stmt, gi
> gcc_assert (TREE_CODE_LENGTH (codecvt1) == unary_op);
> new_temp = make_ssa_name (vec_dest, NULL);
> new_stmt = gimple_build_assign_with_ops (codecvt1,
> - new_temp,
> - vop0, NULL);
> + new_temp, vop0);
> }
>
> vect_finish_stmt_generation (stmt, new_stmt, gsi);
> @@ -3960,7 +3956,7 @@ vectorizable_conversion (gimple stmt, gi
> gcc_assert (TREE_CODE_LENGTH (codecvt1) == unary_op);
> new_temp = make_ssa_name (vec_dest, NULL);
> new_stmt = gimple_build_assign_with_ops (codecvt1, new_temp,
> - vop0, NULL);
> + vop0);
> }
>
> vect_finish_stmt_generation (stmt, new_stmt, gsi);
> @@ -5958,8 +5954,7 @@ vectorizable_load (gimple stmt, gimple_s
> var = make_ssa_name (var, NULL);
> op = build1 (VIEW_CONVERT_EXPR, idxtype, op);
> new_stmt
> - = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var,
> - op, NULL_TREE);
> + = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op);
> vect_finish_stmt_generation (stmt, new_stmt, gsi);
> op = var;
> }
> @@ -5978,8 +5973,7 @@ vectorizable_load (gimple stmt, gimple_s
> var = make_ssa_name (vec_dest, NULL);
> op = build1 (VIEW_CONVERT_EXPR, vectype, op);
> new_stmt
> - = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op,
> - NULL_TREE);
> + = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR, var, op);
> }
> else
> {
> --- gcc/tree-ssa-reassoc.c.jj 2014-11-11 00:06:21.000000000 +0100
> +++ gcc/tree-ssa-reassoc.c 2014-11-19 11:10:22.486663750 +0100
> @@ -3307,7 +3307,7 @@ maybe_optimize_range_tests (gimple stmt)
> }
> else
> g = gimple_build_assign_with_ops (rhs_code, new_lhs,
> - new_op, NULL_TREE);
> + new_op);
> gimple_stmt_iterator gsi = gsi_for_stmt (cast_stmt);
> gimple_set_uid (g, gimple_uid (cast_stmt));
> gimple_set_visited (g, true);
> @@ -4214,10 +4214,11 @@ repropagate_negates (void)
> tree b = gimple_assign_rhs2 (user);
> gimple_stmt_iterator gsi = gsi_for_stmt (feed);
> gimple_stmt_iterator gsi2 = gsi_for_stmt (user);
> - tree x = make_ssa_name (TREE_TYPE (gimple_assign_lhs (feed)), NULL);
> + tree x = make_ssa_name (TREE_TYPE (gimple_assign_lhs (feed)),
> + NULL);
> gimple g = gimple_build_assign_with_ops (PLUS_EXPR, x, a, b);
> gsi_insert_before (&gsi2, g, GSI_SAME_STMT);
> - gimple_assign_set_rhs_with_ops (&gsi2, NEGATE_EXPR, x, NULL);
> + gimple_assign_set_rhs_with_ops (&gsi2, NEGATE_EXPR, x);
> user = gsi_stmt (gsi2);
> update_stmt (user);
> reassoc_remove_stmt (&gsi);
> --- gcc/ubsan.c.jj 2014-11-19 10:22:36.000000000 +0100
> +++ gcc/ubsan.c 2014-11-19 11:21:07.621054607 +0100
> @@ -736,8 +736,7 @@ ubsan_expand_null_ifn (gimple_stmt_itera
> if (compare_tree_int (align, ptralign) == 1)
> {
> check_align = make_ssa_name (pointer_sized_int_node, NULL);
> - g = gimple_build_assign_with_ops (NOP_EXPR, check_align,
> - ptr, NULL_TREE);
> + g = gimple_build_assign_with_ops (NOP_EXPR, check_align, ptr);
> gimple_set_location (g, loc);
> gsi_insert_before (&gsi, g, GSI_SAME_STMT);
> }
> @@ -934,7 +933,7 @@ ubsan_expand_objsize_ifn (gimple_stmt_it
> ? BUILT_IN_UBSAN_HANDLE_TYPE_MISMATCH
> : BUILT_IN_UBSAN_HANDLE_TYPE_MISMATCH_ABORT;
> tree p = make_ssa_name (pointer_sized_int_node, NULL);
> - g = gimple_build_assign_with_ops (NOP_EXPR, p, ptr, NULL_TREE);
> + g = gimple_build_assign_with_ops (NOP_EXPR, p, ptr);
> gimple_set_location (g, loc);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> g = gimple_build_call (builtin_decl_explicit (bcode), 2, data, p);
> @@ -1175,7 +1174,7 @@ instrument_bool_enum_load (gimple_stmt_i
> if (can_throw)
> {
> gimple_assign_set_lhs (stmt, urhs);
> - g = gimple_build_assign_with_ops (NOP_EXPR, lhs, urhs, NULL_TREE);
> + g = gimple_build_assign_with_ops (NOP_EXPR, lhs, urhs);
> gimple_set_location (g, loc);
> edge e = find_fallthru_edge (gimple_bb (stmt)->succs);
> gsi_insert_on_edge_immediate (e, g);
> @@ -1213,7 +1212,7 @@ instrument_bool_enum_load (gimple_stmt_i
>
> if (!can_throw)
> {
> - gimple_assign_set_rhs_with_ops (&gsi2, NOP_EXPR, urhs, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (&gsi2, NOP_EXPR, urhs);
> update_stmt (stmt);
> }
>
> --- gcc/tree-vect-loop.c.jj 2014-11-14 18:27:46.000000000 +0100
> +++ gcc/tree-vect-loop.c 2014-11-19 11:13:02.069792064 +0100
> @@ -2501,7 +2501,7 @@ vect_is_simple_reduction_1 (loop_vec_inf
> tree rhs = gimple_assign_rhs2 (def_stmt);
> tree negrhs = make_ssa_name (TREE_TYPE (rhs), NULL);
> gimple negate_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, negrhs,
> - rhs, NULL);
> + rhs);
> gimple_stmt_iterator gsi = gsi_for_stmt (def_stmt);
> set_vinfo_for_stmt (negate_stmt, new_stmt_vec_info (negate_stmt,
> loop_info, NULL));
> @@ -3379,7 +3379,7 @@ get_initial_def_for_induction (gimple iv
> new_stmt = gimple_build_assign_with_ops
> (VIEW_CONVERT_EXPR,
> vect_get_new_vect_var (vectype, vect_simple_var, "vec_iv_"),
> - build1 (VIEW_CONVERT_EXPR, vectype, vec_init), NULL_TREE);
> + build1 (VIEW_CONVERT_EXPR, vectype, vec_init));
> vec_init = make_ssa_name (gimple_assign_lhs (new_stmt), new_stmt);
> gimple_assign_set_lhs (new_stmt, vec_init);
> new_bb = gsi_insert_on_edge_immediate (loop_preheader_edge (iv_loop),
> @@ -3555,7 +3555,7 @@ get_initial_def_for_induction (gimple iv
> vect_get_new_vect_var (resvectype, vect_simple_var,
> "vec_iv_"),
> build1 (VIEW_CONVERT_EXPR, resvectype,
> - gimple_assign_lhs (new_stmt)), NULL_TREE);
> + gimple_assign_lhs (new_stmt)));
> gimple_assign_set_lhs (new_stmt,
> make_ssa_name
> (gimple_assign_lhs (new_stmt), new_stmt));
> @@ -3622,7 +3622,7 @@ get_initial_def_for_induction (gimple iv
> new_stmt = gimple_build_assign_with_ops
> (VIEW_CONVERT_EXPR,
> vect_get_new_vect_var (resvectype, vect_simple_var, "vec_iv_"),
> - build1 (VIEW_CONVERT_EXPR, resvectype, induc_def), NULL_TREE);
> + build1 (VIEW_CONVERT_EXPR, resvectype, induc_def));
> induc_def = make_ssa_name (gimple_assign_lhs (new_stmt), new_stmt);
> gimple_assign_set_lhs (new_stmt, induc_def);
> si = gsi_after_labels (bb);
> --- gcc/tree-ssa-math-opts.c.jj 2014-11-12 13:10:17.000000000 +0100
> +++ gcc/tree-ssa-math-opts.c 2014-11-19 11:08:43.533444405 +0100
> @@ -1129,7 +1129,7 @@ build_and_insert_cast (gimple_stmt_itera
> tree type, tree val)
> {
> tree result = make_ssa_name (type, NULL);
> - gimple stmt = gimple_build_assign_with_ops (NOP_EXPR, result, val, NULL_TREE);
> + gimple stmt = gimple_build_assign_with_ops (NOP_EXPR, result, val);
> gimple_set_location (stmt, loc);
> gsi_insert_before (gsi, stmt, GSI_SAME_STMT);
> return result;
> @@ -2259,13 +2259,11 @@ bswap_replace (gimple cur_stmt, gimple s
> load_stmt = gimple_build_assign (val_tmp, val_expr);
> gimple_set_vuse (load_stmt, n->vuse);
> gsi_insert_before (&gsi, load_stmt, GSI_SAME_STMT);
> - gimple_assign_set_rhs_with_ops_1 (&gsi, NOP_EXPR, val_tmp,
> - NULL_TREE, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (&gsi, NOP_EXPR, val_tmp);
> }
> else
> {
> - gimple_assign_set_rhs_with_ops_1 (&gsi, MEM_REF, val_expr,
> - NULL_TREE, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (&gsi, MEM_REF, val_expr);
> gimple_set_vuse (cur_stmt, n->vuse);
> }
> update_stmt (cur_stmt);
> @@ -2319,8 +2317,7 @@ bswap_replace (gimple cur_stmt, gimple s
> {
> gimple convert_stmt;
> tmp = make_temp_ssa_name (bswap_type, NULL, "bswapsrc");
> - convert_stmt = gimple_build_assign_with_ops (NOP_EXPR, tmp, src,
> - NULL);
> + convert_stmt = gimple_build_assign_with_ops (NOP_EXPR, tmp, src);
> gsi_insert_before (&gsi, convert_stmt, GSI_SAME_STMT);
> }
>
> @@ -2334,7 +2331,7 @@ bswap_replace (gimple cur_stmt, gimple s
> {
> gimple convert_stmt;
> tmp = make_temp_ssa_name (bswap_type, NULL, "bswapdst");
> - convert_stmt = gimple_build_assign_with_ops (NOP_EXPR, tgt, tmp, NULL);
> + convert_stmt = gimple_build_assign_with_ops (NOP_EXPR, tgt, tmp);
> gsi_insert_after (&gsi, convert_stmt, GSI_SAME_STMT);
> }
>
> @@ -2973,8 +2970,8 @@ convert_plusminus_to_widen (gimple_stmt_
> if (TREE_CODE (mult_rhs2) == INTEGER_CST)
> mult_rhs2 = fold_convert (type2, mult_rhs2);
>
> - gimple_assign_set_rhs_with_ops_1 (gsi, wmult_code, mult_rhs1, mult_rhs2,
> - add_rhs);
> + gimple_assign_set_rhs_with_ops (gsi, wmult_code, mult_rhs1, mult_rhs2,
> + add_rhs);
> update_stmt (gsi_stmt (*gsi));
> widen_mul_stats.maccs_inserted++;
> return true;
> @@ -3177,8 +3174,7 @@ convert_mult_to_fma (gimple mul_stmt, tr
>
> fma_stmt = gimple_build_assign_with_ops (FMA_EXPR,
> gimple_assign_lhs (use_stmt),
> - mulop1, op2,
> - addop);
> + mulop1, op2, addop);
> gsi_replace (&gsi, fma_stmt, true);
> widen_mul_stats.fmas_inserted++;
> }
> --- gcc/tree-loop-distribution.c.jj 2014-10-28 14:39:56.000000000 +0100
> +++ gcc/tree-loop-distribution.c 2014-11-19 11:06:07.496252282 +0100
> @@ -821,9 +821,8 @@ generate_memset_builtin (struct loop *lo
> val = fold_convert (integer_type_node, val);
> else if (!useless_type_conversion_p (integer_type_node, TREE_TYPE (val)))
> {
> - gimple cstmt;
> tree tem = make_ssa_name (integer_type_node, NULL);
> - cstmt = gimple_build_assign_with_ops (NOP_EXPR, tem, val, NULL_TREE);
> + gimple cstmt = gimple_build_assign_with_ops (NOP_EXPR, tem, val);
> gsi_insert_after (&gsi, cstmt, GSI_CONTINUE_LINKING);
> val = tem;
> }
> --- gcc/tree-vect-patterns.c.jj 2014-11-11 00:06:21.000000000 +0100
> +++ gcc/tree-vect-patterns.c 2014-11-19 11:16:05.138497757 +0100
> @@ -760,8 +760,7 @@ vect_handle_widen_op_by_const (gimple st
> /* Create a_T = (NEW_TYPE) a_t; */
> *oprnd = gimple_assign_rhs1 (def_stmt);
> new_oprnd = make_ssa_name (new_type, NULL);
> - new_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd, *oprnd,
> - NULL_TREE);
> + new_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd, *oprnd);
> STMT_VINFO_RELATED_STMT (vinfo_for_stmt (def_stmt)) = new_stmt;
> stmts->safe_push (def_stmt);
> *oprnd = new_oprnd;
> @@ -935,7 +934,7 @@ vect_recog_widen_mult_pattern (vec<gimpl
> tree old_oprnd = gimple_assign_rhs1 (def_stmt);
> tree new_oprnd = make_ssa_name (half_type0, NULL);
> new_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd,
> - old_oprnd, NULL_TREE);
> + old_oprnd);
> *oprnd = new_oprnd;
> }
>
> @@ -1032,8 +1031,7 @@ vect_recog_widen_mult_pattern (vec<gimpl
> pattern_stmt
> = gimple_build_assign_with_ops (NOP_EXPR,
> vect_recog_temp_ssa_var (type, NULL),
> - gimple_assign_lhs (pattern_stmt),
> - NULL_TREE);
> + gimple_assign_lhs (pattern_stmt));
> }
>
> if (dump_enabled_p ())
> @@ -1443,7 +1441,7 @@ vect_operation_fits_smaller_type (gimple
> oprnd = gimple_assign_rhs1 (def_stmt);
> new_oprnd = make_ssa_name (interm_type, NULL);
> new_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd,
> - oprnd, NULL_TREE);
> + oprnd);
> STMT_VINFO_RELATED_STMT (vinfo_for_stmt (def_stmt)) = new_stmt;
> stmts->safe_push (def_stmt);
> oprnd = new_oprnd;
> @@ -1462,7 +1460,7 @@ vect_operation_fits_smaller_type (gimple
> /* Create a type conversion HALF_TYPE->INTERM_TYPE. */
> new_oprnd = make_ssa_name (interm_type, NULL);
> new_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd,
> - oprnd, NULL_TREE);
> + oprnd);
> oprnd = new_oprnd;
> *new_def_stmt = new_stmt;
> }
> @@ -1594,7 +1592,7 @@ vect_recog_over_widening_pattern (vec<gi
> /* Create NEW_TYPE->USE_TYPE conversion. */
> new_oprnd = make_ssa_name (use_type, NULL);
> pattern_stmt = gimple_build_assign_with_ops (NOP_EXPR, new_oprnd,
> - var, NULL_TREE);
> + var);
> STMT_VINFO_RELATED_STMT (vinfo_for_stmt (use_stmt)) = pattern_stmt;
>
> *type_in = get_vectype_for_scalar_type (new_type);
> @@ -1946,8 +1944,7 @@ vect_recog_rotate_pattern (vec<gimple> *
> if (def == NULL_TREE)
> {
> def = vect_recog_temp_ssa_var (type, NULL);
> - def_stmt = gimple_build_assign_with_ops (NOP_EXPR, def, oprnd1,
> - NULL_TREE);
> + def_stmt = gimple_build_assign_with_ops (NOP_EXPR, def, oprnd1);
> if (ext_def)
> {
> basic_block new_bb
> @@ -1977,8 +1974,7 @@ vect_recog_rotate_pattern (vec<gimple> *
> if (vecstype == NULL_TREE)
> return NULL;
> def2 = vect_recog_temp_ssa_var (stype, NULL);
> - def_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, def2, def,
> - NULL_TREE);
> + def_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, def2, def);
> if (ext_def)
> {
> basic_block new_bb
> @@ -2151,8 +2147,7 @@ vect_recog_vector_vector_shift_pattern (
> if (def == NULL_TREE)
> {
> def = vect_recog_temp_ssa_var (TREE_TYPE (oprnd0), NULL);
> - def_stmt = gimple_build_assign_with_ops (NOP_EXPR, def, oprnd1,
> - NULL_TREE);
> + def_stmt = gimple_build_assign_with_ops (NOP_EXPR, def, oprnd1);
> new_pattern_def_seq (stmt_vinfo, def_stmt);
> }
>
> @@ -2350,8 +2345,7 @@ vect_recog_divmod_pattern (vec<gimple> *
> append_pattern_def_seq (stmt_vinfo, def_stmt);
> signmask = vect_recog_temp_ssa_var (itype, NULL);
> def_stmt
> - = gimple_build_assign_with_ops (NOP_EXPR, signmask, var,
> - NULL_TREE);
> + = gimple_build_assign_with_ops (NOP_EXPR, signmask, var);
> append_pattern_def_seq (stmt_vinfo, def_stmt);
> }
> def_stmt
> @@ -2613,8 +2607,7 @@ vect_recog_divmod_pattern (vec<gimple> *
> if (msb != 1)
> def_stmt
> = gimple_build_assign_with_ops (INTEGER_CST,
> - t4, build_int_cst (itype, msb),
> - NULL_TREE);
> + t4, build_int_cst (itype, msb));
> else
> def_stmt
> = gimple_build_assign_with_ops (RSHIFT_EXPR, t4, oprnd0,
> @@ -2809,7 +2802,7 @@ vect_recog_mixed_size_cond_pattern (vec<
> pattern_stmt
> = gimple_build_assign_with_ops (NOP_EXPR,
> vect_recog_temp_ssa_var (type, NULL),
> - gimple_assign_lhs (def_stmt), NULL_TREE);
> + gimple_assign_lhs (def_stmt));
>
> new_pattern_def_seq (stmt_vinfo, def_stmt);
> def_stmt_info = new_stmt_vec_info (def_stmt, loop_vinfo, bb_vinfo);
> @@ -2923,8 +2916,7 @@ adjust_bool_pattern_cast (tree type, tre
> cast_stmt
> = gimple_build_assign_with_ops (NOP_EXPR,
> vect_recog_temp_ssa_var (type, NULL),
> - gimple_assign_lhs (pattern_stmt),
> - NULL_TREE);
> + gimple_assign_lhs (pattern_stmt));
> STMT_VINFO_RELATED_STMT (stmt_vinfo) = cast_stmt;
> return gimple_assign_lhs (cast_stmt);
> }
> @@ -2961,7 +2953,7 @@ adjust_bool_pattern (tree var, tree out_
> pattern_stmt
> = gimple_build_assign_with_ops (SSA_NAME,
> vect_recog_temp_ssa_var (itype, NULL),
> - irhs1, NULL_TREE);
> + irhs1);
> break;
>
> case BIT_NOT_EXPR:
> @@ -3209,10 +3201,10 @@ vect_recog_bool_pattern (vec<gimple> *st
> lhs = vect_recog_temp_ssa_var (TREE_TYPE (lhs), NULL);
> if (useless_type_conversion_p (TREE_TYPE (lhs), TREE_TYPE (rhs)))
> pattern_stmt
> - = gimple_build_assign_with_ops (SSA_NAME, lhs, rhs, NULL_TREE);
> + = gimple_build_assign_with_ops (SSA_NAME, lhs, rhs);
> else
> pattern_stmt
> - = gimple_build_assign_with_ops (NOP_EXPR, lhs, rhs, NULL_TREE);
> + = gimple_build_assign_with_ops (NOP_EXPR, lhs, rhs);
> *type_out = vectype;
> *type_in = vectype;
> stmts->safe_push (last_stmt);
> @@ -3278,12 +3270,12 @@ vect_recog_bool_pattern (vec<gimple> *st
> {
> tree rhs2 = vect_recog_temp_ssa_var (TREE_TYPE (lhs), NULL);
> gimple cast_stmt
> - = gimple_build_assign_with_ops (NOP_EXPR, rhs2, rhs, NULL_TREE);
> + = gimple_build_assign_with_ops (NOP_EXPR, rhs2, rhs);
> new_pattern_def_seq (stmt_vinfo, cast_stmt);
> rhs = rhs2;
> }
> pattern_stmt
> - = gimple_build_assign_with_ops (SSA_NAME, lhs, rhs, NULL_TREE);
> + = gimple_build_assign_with_ops (SSA_NAME, lhs, rhs);
> pattern_stmt_info = new_stmt_vec_info (pattern_stmt, loop_vinfo,
> bb_vinfo);
> set_vinfo_for_stmt (pattern_stmt, pattern_stmt_info);
> --- gcc/tree-ssa-phiopt.c.jj 2014-11-13 12:50:47.000000000 +0100
> +++ gcc/tree-ssa-phiopt.c 2014-11-19 11:09:33.025553799 +0100
> @@ -536,7 +536,7 @@ conditional_replacement (basic_block con
>
> new_var2 = make_ssa_name (TREE_TYPE (result), NULL);
> new_stmt = gimple_build_assign_with_ops (CONVERT_EXPR, new_var2,
> - new_var, NULL);
> + new_var);
> gsi_insert_before (&gsi, new_stmt, GSI_SAME_STMT);
> new_var = new_var2;
>
> @@ -1254,7 +1254,7 @@ abs_replacement (basic_block cond_bb, ba
> lhs = result;
>
> /* Build the modify expression with abs expression. */
> - new_stmt = gimple_build_assign_with_ops (ABS_EXPR, lhs, rhs, NULL);
> + new_stmt = gimple_build_assign_with_ops (ABS_EXPR, lhs, rhs);
>
> gsi = gsi_last_bb (cond_bb);
> gsi_insert_before (&gsi, new_stmt, GSI_NEW_STMT);
> @@ -1264,7 +1264,7 @@ abs_replacement (basic_block cond_bb, ba
> /* Get the right GSI. We want to insert after the recently
> added ABS_EXPR statement (which we know is the first statement
> in the block. */
> - new_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, result, lhs, NULL);
> + new_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, result, lhs);
>
> gsi_insert_after (&gsi, new_stmt, GSI_NEW_STMT);
> }
> @@ -1388,12 +1388,12 @@ neg_replacement (basic_block cond_bb, ba
> logical and arithmetic operations on it. */
> tree cond_val_converted = make_ssa_name (TREE_TYPE (rhs), NULL);
> new_stmt = gimple_build_assign_with_ops (NOP_EXPR, cond_val_converted,
> - cond_val, NULL_TREE);
> + cond_val);
> gsi_insert_after (&gsi, new_stmt, GSI_NEW_STMT);
>
> tree neg_cond_val_converted = make_ssa_name (TREE_TYPE (rhs), NULL);
> new_stmt = gimple_build_assign_with_ops (NEGATE_EXPR, neg_cond_val_converted,
> - cond_val_converted, NULL_TREE);
> + cond_val_converted);
> gsi_insert_after (&gsi, new_stmt, GSI_NEW_STMT);
>
> tree tmp = make_ssa_name (TREE_TYPE (rhs), NULL);
> --- gcc/asan.c.jj 2014-11-18 23:10:56.000000000 +0100
> +++ gcc/asan.c 2014-11-19 10:59:53.002995844 +0100
> @@ -1599,15 +1599,14 @@ build_shadow_mem_access (gimple_stmt_ite
>
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (shadow_ptr_type, NULL),
> - gimple_assign_lhs (g), NULL_TREE);
> + gimple_assign_lhs (g));
> gimple_set_location (g, location);
> gsi_insert_after (gsi, g, GSI_NEW_STMT);
>
> t = build2 (MEM_REF, shadow_type, gimple_assign_lhs (g),
> build_int_cst (shadow_ptr_type, 0));
> g = gimple_build_assign_with_ops (MEM_REF,
> - make_ssa_name (shadow_type, NULL),
> - t, NULL_TREE);
> + make_ssa_name (shadow_type, NULL), t);
> gimple_set_location (g, location);
> gsi_insert_after (gsi, g, GSI_NEW_STMT);
> return gimple_assign_lhs (g);
> @@ -1625,7 +1624,7 @@ maybe_create_ssa_name (location_t loc, t
> gimple g
> = gimple_build_assign_with_ops (TREE_CODE (base),
> make_ssa_name (TREE_TYPE (base), NULL),
> - base, NULL_TREE);
> + base);
> gimple_set_location (g, loc);
> if (before_p)
> gsi_insert_before (iter, g, GSI_SAME_STMT);
> @@ -1645,8 +1644,8 @@ maybe_cast_to_ptrmode (location_t loc, t
> return len;
> gimple g
> = gimple_build_assign_with_ops (NOP_EXPR,
> - make_ssa_name (pointer_sized_int_node, NULL),
> - len, NULL);
> + make_ssa_name (pointer_sized_int_node,
> + NULL), len);
> gimple_set_location (g, loc);
> if (before_p)
> gsi_insert_before (iter, g, GSI_SAME_STMT);
> @@ -2536,8 +2535,7 @@ asan_expand_check_ifn (gimple_stmt_itera
> gimple g
> = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (pointer_sized_int_node,
> - NULL),
> - base, NULL_TREE);
> + NULL), base);
> gimple_set_location (g, loc);
> gsi_insert_before (iter, g, GSI_SAME_STMT);
> tree base_addr = gimple_assign_lhs (g);
> @@ -2551,8 +2549,7 @@ asan_expand_check_ifn (gimple_stmt_itera
> gcc_assert (nargs == 2);
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (pointer_sized_int_node,
> - NULL),
> - len, NULL_TREE);
> + NULL), len);
> gimple_set_location (g, loc);
> gsi_insert_before (iter, g, GSI_SAME_STMT);
> tree sz_arg = gimple_assign_lhs (g);
> @@ -2610,8 +2607,7 @@ asan_expand_check_ifn (gimple_stmt_itera
>
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (pointer_sized_int_node,
> - NULL),
> - base, NULL_TREE);
> + NULL), base);
> gimple_set_location (g, loc);
> gsi_insert_before (&gsi, g, GSI_NEW_STMT);
> tree base_addr = gimple_assign_lhs (g);
> --- gcc/tree-vect-slp.c.jj 2014-11-11 00:06:25.000000000 +0100
> +++ gcc/tree-vect-slp.c 2014-11-19 11:16:29.618057250 +0100
> @@ -2609,7 +2609,7 @@ vect_get_constant_vectors (tree op, slp_
> op);
> init_stmt
> = gimple_build_assign_with_ops (VIEW_CONVERT_EXPR,
> - new_temp, op, NULL_TREE);
> + new_temp, op);
> gimple_seq_add_stmt (&ctor_seq, init_stmt);
> op = new_temp;
> }
> --- gcc/omp-low.c.jj 2014-11-19 10:48:20.000000000 +0100
> +++ gcc/omp-low.c 2014-11-19 11:04:41.052808846 +0100
> @@ -3839,8 +3839,7 @@ lower_rec_input_clauses (tree clauses, g
> OMP_CLAUSE_CHAIN (c) = gimple_omp_for_clauses (ctx->stmt);
> gimple_omp_for_set_clauses (ctx->stmt, c);
> g = gimple_build_assign_with_ops (INTEGER_CST, lane,
> - build_int_cst (unsigned_type_node, 0),
> - NULL_TREE);
> + build_int_cst (unsigned_type_node, 0));
> gimple_seq_add_stmt (ilist, g);
> for (int i = 0; i < 2; i++)
> if (llist[i])
> @@ -3851,7 +3850,7 @@ lower_rec_input_clauses (tree clauses, g
> gimple_seq *seq = i == 0 ? ilist : dlist;
> gimple_seq_add_stmt (seq, g);
> tree t = build_int_cst (unsigned_type_node, 0);
> - g = gimple_build_assign_with_ops (INTEGER_CST, idx, t, NULL_TREE);
> + g = gimple_build_assign_with_ops (INTEGER_CST, idx, t);
> gimple_seq_add_stmt (seq, g);
> tree body = create_artificial_label (UNKNOWN_LOCATION);
> tree header = create_artificial_label (UNKNOWN_LOCATION);
> @@ -5845,8 +5844,7 @@ expand_omp_for_generic (struct omp_regio
> if (useless_type_conversion_p (TREE_TYPE (fd->loop.v), TREE_TYPE (iend)))
> stmt = gimple_build_assign (fd->loop.v, iend);
> else
> - stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, iend,
> - NULL_TREE);
> + stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, iend);
> gsi_insert_after (&gsi, stmt, GSI_CONTINUE_LINKING);
> }
> if (fd->collapse > 1)
> @@ -6266,8 +6264,7 @@ expand_omp_for_static_nochunk (struct om
> if (useless_type_conversion_p (TREE_TYPE (fd->loop.v), TREE_TYPE (e)))
> stmt = gimple_build_assign (fd->loop.v, e);
> else
> - stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, e,
> - NULL_TREE);
> + stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, e);
> gsi_insert_after (&gsi, stmt, GSI_CONTINUE_LINKING);
> }
> if (fd->collapse > 1)
> @@ -6657,8 +6654,7 @@ expand_omp_for_static_chunk (struct omp_
> if (useless_type_conversion_p (TREE_TYPE (fd->loop.v), TREE_TYPE (e)))
> stmt = gimple_build_assign (fd->loop.v, e);
> else
> - stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, e,
> - NULL_TREE);
> + stmt = gimple_build_assign_with_ops (NOP_EXPR, fd->loop.v, e);
> gsi_insert_after (&gsi, stmt, GSI_CONTINUE_LINKING);
> }
> if (fd->collapse > 1)
> @@ -12198,8 +12194,7 @@ simd_clone_adjust (struct cgraph_node *n
> {
> t = make_ssa_name (orig_arg, NULL);
> g = gimple_build_assign_with_ops (NOP_EXPR, t,
> - gimple_call_lhs (g),
> - NULL_TREE);
> + gimple_call_lhs (g));
> gimple_seq_add_stmt_without_update (&seq, g);
> }
> gsi_insert_seq_on_edge_immediate
> --- gcc/tree-ssa-strlen.c.jj 2014-11-18 08:26:38.000000000 +0100
> +++ gcc/tree-ssa-strlen.c 2014-11-19 11:10:51.596139927 +0100
> @@ -1816,7 +1816,7 @@ handle_pointer_plus (gimple_stmt_iterato
> enum tree_code rhs_code
> = useless_type_conversion_p (TREE_TYPE (lhs), TREE_TYPE (si->endptr))
> ? SSA_NAME : NOP_EXPR;
> - gimple_assign_set_rhs_with_ops (gsi, rhs_code, si->endptr, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (gsi, rhs_code, si->endptr);
> gcc_assert (gsi_stmt (*gsi) == stmt);
> update_stmt (stmt);
> }
> --- gcc/gimple-fold.c.jj 2014-11-14 15:11:49.000000000 +0100
> +++ gcc/gimple-fold.c 2014-11-19 11:01:57.211759180 +0100
> @@ -2962,8 +2962,7 @@ replace_stmt_with_simplification (gimple
> maybe_build_generic_op (rcode,
> TREE_TYPE (gimple_assign_lhs (stmt)),
> &ops[0], ops[1], ops[2]);
> - gimple_assign_set_rhs_with_ops_1 (gsi, rcode,
> - ops[0], ops[1], ops[2]);
> + gimple_assign_set_rhs_with_ops (gsi, rcode, ops[0], ops[1], ops[2]);
> if (dump_file && (dump_flags & TDF_DETAILS))
> {
> fprintf (dump_file, "gimple_simplified to ");
> @@ -5623,8 +5622,8 @@ rewrite_to_defined_overflow (gimple stmt
> if (gimple_assign_rhs_code (stmt) == POINTER_PLUS_EXPR)
> gimple_assign_set_rhs_code (stmt, PLUS_EXPR);
> gimple_seq_add_stmt (&stmts, stmt);
> - gimple cvt = gimple_build_assign_with_ops
> - (NOP_EXPR, lhs, gimple_assign_lhs (stmt), NULL_TREE);
> + gimple cvt = gimple_build_assign_with_ops (NOP_EXPR, lhs,
> + gimple_assign_lhs (stmt));
> gimple_seq_add_stmt (&stmts, cvt);
>
> return stmts;
> @@ -5654,10 +5653,9 @@ gimple_build (gimple_seq *seq, location_
> || code == IMAGPART_EXPR
> || code == VIEW_CONVERT_EXPR)
> stmt = gimple_build_assign_with_ops (code, res,
> - build1 (code, type,
> - op0), NULL_TREE);
> + build1 (code, type, op0));
> else
> - stmt = gimple_build_assign_with_ops (code, res, op0, NULL_TREE);
> + stmt = gimple_build_assign_with_ops (code, res, op0);
> gimple_set_location (stmt, loc);
> gimple_seq_add_stmt_without_update (seq, stmt);
> }
> @@ -5712,8 +5710,7 @@ gimple_build (gimple_seq *seq, location_
> if (code == BIT_FIELD_REF)
> stmt = gimple_build_assign_with_ops (code, res,
> build3 (BIT_FIELD_REF, type,
> - op0, op1, op2),
> - NULL_TREE);
> + op0, op1, op2));
> else
> stmt = gimple_build_assign_with_ops (code, res, op0, op1, op2);
> gimple_set_location (stmt, loc);
> --- gcc/tree-vect-loop-manip.c.jj 2014-10-28 14:40:00.000000000 +0100
> +++ gcc/tree-vect-loop-manip.c 2014-11-19 11:13:22.357426990 +0100
> @@ -2157,7 +2157,7 @@ vect_create_cond_for_align_checks (loop_
> sprintf (tmp_name, "addr2int%d", i);
> addr_tmp_name = make_temp_ssa_name (int_ptrsize_type, NULL, tmp_name);
> addr_stmt = gimple_build_assign_with_ops (NOP_EXPR, addr_tmp_name,
> - addr_base, NULL_TREE);
> + addr_base);
> gimple_seq_add_stmt (cond_expr_stmt_list, addr_stmt);
>
> /* The addresses are OR together. */
> --- gcc/gimple-ssa-strength-reduction.c.jj 2014-11-11 00:06:18.000000000 +0100
> +++ gcc/gimple-ssa-strength-reduction.c 2014-11-19 11:03:10.675436298 +0100
> @@ -3261,8 +3261,7 @@ introduce_cast_before_cand (slsr_cand_t
> gimple_stmt_iterator gsi = gsi_for_stmt (c->cand_stmt);
>
> cast_lhs = make_temp_ssa_name (to_type, NULL, "slsr");
> - cast_stmt = gimple_build_assign_with_ops (NOP_EXPR, cast_lhs,
> - from_expr, NULL_TREE);
> + cast_stmt = gimple_build_assign_with_ops (NOP_EXPR, cast_lhs, from_expr);
> gimple_set_location (cast_stmt, gimple_location (c->cand_stmt));
> gsi_insert_before (&gsi, cast_stmt, GSI_SAME_STMT);
>
> @@ -3432,8 +3431,7 @@ replace_one_candidate (slsr_cand_t c, un
> {
> gimple_stmt_iterator gsi = gsi_for_stmt (c->cand_stmt);
> gimple cast_stmt = gimple_build_assign_with_ops (NOP_EXPR, lhs,
> - basis_name,
> - NULL_TREE);
> + basis_name);
> gimple_set_location (cast_stmt, gimple_location (c->cand_stmt));
> gsi_replace (&gsi, cast_stmt, false);
> c->cand_stmt = cast_stmt;
> --- gcc/gimple-builder.c.jj 2014-10-28 14:40:20.000000000 +0100
> +++ gcc/gimple-builder.c 2014-11-19 11:00:26.613390612 +0100
> @@ -120,7 +120,7 @@ build_type_cast (tree to_type, tree op,
> {
> if (lhs == NULL_TREE)
> lhs = make_ssa_name (to_type, NULL);
> - return gimple_build_assign_with_ops (NOP_EXPR, lhs, op, NULL_TREE);
> + return gimple_build_assign_with_ops (NOP_EXPR, lhs, op);
> }
>
> gimple
> --- gcc/tree-vrp.c.jj 2014-11-14 15:11:48.000000000 +0100
> +++ gcc/tree-vrp.c 2014-11-19 11:19:55.120359252 +0100
> @@ -8879,15 +8879,14 @@ simplify_truth_ops_using_ranges (gimple_
> if (integer_zerop (op1))
> gimple_assign_set_rhs_with_ops (gsi,
> need_conversion
> - ? NOP_EXPR : TREE_CODE (op0),
> - op0, NULL_TREE);
> + ? NOP_EXPR : TREE_CODE (op0), op0);
> /* For A != B we substitute A ^ B. Either with conversion. */
> else if (need_conversion)
> {
> tree tem = make_ssa_name (TREE_TYPE (op0), NULL);
> gimple newop = gimple_build_assign_with_ops (BIT_XOR_EXPR, tem, op0, op1);
> gsi_insert_before (gsi, newop, GSI_SAME_STMT);
> - gimple_assign_set_rhs_with_ops (gsi, NOP_EXPR, tem, NULL_TREE);
> + gimple_assign_set_rhs_with_ops (gsi, NOP_EXPR, tem);
> }
> /* Or without. */
> else
> @@ -9107,7 +9106,7 @@ simplify_bit_ops_using_ranges (gimple_st
> if (op == NULL_TREE)
> return false;
>
> - gimple_assign_set_rhs_with_ops (gsi, TREE_CODE (op), op, NULL);
> + gimple_assign_set_rhs_with_ops (gsi, TREE_CODE (op), op);
> update_stmt (gsi_stmt (*gsi));
> return true;
> }
> @@ -9611,7 +9610,7 @@ simplify_float_conversion_using_ranges (
> float conversion. */
> tem = make_ssa_name (build_nonstandard_integer_type
> (GET_MODE_PRECISION (mode), 0), NULL);
> - conv = gimple_build_assign_with_ops (NOP_EXPR, tem, rhs1, NULL_TREE);
> + conv = gimple_build_assign_with_ops (NOP_EXPR, tem, rhs1);
> gsi_insert_before (gsi, conv, GSI_SAME_STMT);
> gimple_assign_set_rhs1 (stmt, tem);
> update_stmt (stmt);
> @@ -9685,8 +9684,7 @@ simplify_internal_call_using_ranges (gim
> else if (!useless_type_conversion_p (utype, TREE_TYPE (op0)))
> {
> g = gimple_build_assign_with_ops (NOP_EXPR,
> - make_ssa_name (utype, NULL),
> - op0, NULL_TREE);
> + make_ssa_name (utype, NULL), op0);
> gimple_set_location (g, loc);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> op0 = gimple_assign_lhs (g);
> @@ -9696,8 +9694,7 @@ simplify_internal_call_using_ranges (gim
> else if (!useless_type_conversion_p (utype, TREE_TYPE (op1)))
> {
> g = gimple_build_assign_with_ops (NOP_EXPR,
> - make_ssa_name (utype, NULL),
> - op1, NULL_TREE);
> + make_ssa_name (utype, NULL), op1);
> gimple_set_location (g, loc);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> op1 = gimple_assign_lhs (g);
> @@ -9710,7 +9707,7 @@ simplify_internal_call_using_ranges (gim
> {
> g = gimple_build_assign_with_ops (NOP_EXPR,
> make_ssa_name (type, NULL),
> - gimple_assign_lhs (g), NULL_TREE);
> + gimple_assign_lhs (g));
> gimple_set_location (g, loc);
> gsi_insert_before (gsi, g, GSI_SAME_STMT);
> }
>
> Jakub
>
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendoerffer, HRB 21284
(AG Nuernberg)
Maxfeldstrasse 5, 90409 Nuernberg, Germany
More information about the Gcc-patches
mailing list