From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 7879) id B63313858406; Fri, 2 Sep 2022 15:31:42 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org B63313858406 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1662132702; bh=mYQtUesVxurIvsmVukTFV+FCjXH1j3YbekAQkDPpLog=; h=From:To:Subject:Date:From; b=EokvWbzNVkhUzt9QNGcRQGUsB+VYFnNltbM7lM+2duSydzda3VXNXD+PNDepfxlBN pb2lGDK1w3FIWKvtBmU2EGRFBggLRauMtZL6U2Hje6s05rkHNoGK/PYTtQapJQEE+T nb3OAqpqBwI5aVw2cREmWI/WvdW4B+Kl8Hrgr5qM= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Filip Kastl To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/pheeck/heads/sccp)] sccp now actually replaces phis with values X-Act-Checkin: gcc X-Git-Author: Filip Kastl X-Git-Refname: refs/users/pheeck/heads/sccp X-Git-Oldrev: d971e7347969e6ad1956fac4fafc5dc59ecdd4e4 X-Git-Newrev: c798727d40ceb10337f785c448d07e62c184513c Message-Id: <20220902153142.B63313858406@sourceware.org> Date: Fri, 2 Sep 2022 15:31:42 +0000 (GMT) List-Id: https://gcc.gnu.org/g:c798727d40ceb10337f785c448d07e62c184513c commit c798727d40ceb10337f785c448d07e62c184513c Author: Filip Kastl Date: Fri Sep 2 17:31:36 2022 +0200 sccp now actually replaces phis with values Diff: --- gcc/sccp.cc | 36 +++++++++++++++++++++++++++++++++--- 1 file changed, 33 insertions(+), 3 deletions(-) diff --git a/gcc/sccp.cc b/gcc/sccp.cc index 8b57e3a4a3c..a4b332e0c29 100644 --- a/gcc/sccp.cc +++ b/gcc/sccp.cc @@ -320,14 +320,43 @@ replace_scc_by_value (vec scc, tree v) { for (gphi *phi : scc) { - // DEBUG tree ssa_name = gimple_get_lhs (phi); + + // DEBUG unsigned vnum_get_replaced = SSA_NAME_VERSION (ssa_name); unsigned vnum_replaced_by = SSA_NAME_VERSION (v); std::cerr << "Replacing " << vnum_get_replaced << " by " << vnum_replaced_by << std::endl; - // TODO Remove phi statement and free ssa name - // TODO Replace occurences of phi with v + + /* Replace each occurence of phi by value v. */ + use_operand_p use_p; + imm_use_iterator iter; + gimple *use_stmt; + FOR_EACH_IMM_USE_STMT (use_stmt, iter, ssa_name) + FOR_EACH_IMM_USE_ON_STMT (use_p, iter) + SET_USE (use_p, v); + } +} + +/* Remove all PHIs with zero uses. */ + +static void +remove_zero_uses_phis () +{ + basic_block bb; + FOR_EACH_BB_FN (bb, cfun) + { + gphi_iterator pi; + for (pi = gsi_start_phis (bb); !gsi_end_p (pi);) + { + gphi *phi = pi.phi (); + tree ssa_name = gimple_phi_result (phi); + if (has_zero_uses (ssa_name)) + /* Note that remove_phi_node() also frees SSA name. */ + remove_phi_node (&pi, true); + else + gsi_next (&pi); + } } } @@ -393,6 +422,7 @@ remove_redundant_phis (vec phis) { process_scc (scc); } + remove_zero_uses_phis (); } /* TODO Pass description. */