From: Richard Biener Date: Fri, 14 Dec 2018 09:26:05 +0000 (+0000) Subject: gimple-ssa-split-paths.c (is_feasible_trace): Remove duplicated code block. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b210534e5c630eda94b210a8d22dbfae1299d2da;p=gcc.git gimple-ssa-split-paths.c (is_feasible_trace): Remove duplicated code block. 2018-12-14 Richard Biener * gimple-ssa-split-paths.c (is_feasible_trace): Remove duplicated code block. From-SVN: r267126 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index aa260995e8d..5d81eb0beb9 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-12-14 Richard Biener + + * gimple-ssa-split-paths.c (is_feasible_trace): Remove + duplicated code block. + 2018-12-14 Alan Modra PR rtl-optimization/88311 diff --git a/gcc/gimple-ssa-split-paths.c b/gcc/gimple-ssa-split-paths.c index 91596526045..75f6be8f1cf 100644 --- a/gcc/gimple-ssa-split-paths.c +++ b/gcc/gimple-ssa-split-paths.c @@ -249,52 +249,6 @@ is_feasible_trace (basic_block bb) } } - /* Canonicalize the form. */ - if (num_stmts_in_pred1 == 0 && num_stmts_in_pred2 == 1) - { - std::swap (pred1, pred2); - std::swap (num_stmts_in_pred1, num_stmts_in_pred2); - } - - /* Another variant. This one is half-diamond. */ - if (num_stmts_in_pred1 == 1 && num_stmts_in_pred2 == 0 - && dominated_by_p (CDI_DOMINATORS, pred1, pred2)) - { - gimple *stmt1 = last_and_only_stmt (pred1); - - /* The only statement in PRED1 must be an assignment that is - not a good candidate for if-conversion. This may need some - generalization. */ - if (stmt1 && gimple_code (stmt1) == GIMPLE_ASSIGN) - { - enum tree_code code1 = gimple_assign_rhs_code (stmt1); - - if (!poor_ifcvt_candidate_code (code1)) - { - tree lhs1 = gimple_assign_lhs (stmt1); - tree rhs1 = gimple_assign_rhs1 (stmt1); - - gimple_stmt_iterator gsi; - for (gsi = gsi_start_phis (bb); !gsi_end_p (gsi); gsi_next (&gsi)) - { - gimple *phi = gsi_stmt (gsi); - if ((gimple_phi_arg_def (phi, 0) == lhs1 - && gimple_phi_arg_def (phi, 1) == rhs1) - || (gimple_phi_arg_def (phi, 1) == lhs1 - && gimple_phi_arg_def (phi, 0) == rhs1)) - { - if (dump_file && (dump_flags & TDF_DETAILS)) - fprintf (dump_file, - "Block %d appears to be a join point for " - "if-convertable half-diamond.\n", - bb->index); - return false; - } - } - } - } - } - /* If the joiner has no PHIs with useful uses there is zero chance of CSE/DCE/jump-threading possibilities exposed by duplicating it. */ bool found_useful_phi = false;