[gcc(refs/users/pheeck/heads/sccp)] added call to update_gimple_stmt ()
Filip Kastl
pheeck@gcc.gnu.org
Wed Sep 7 09:33:18 GMT 2022
https://gcc.gnu.org/g:d0786c70d75b78a35e4cf723be921ccf09657bec
commit d0786c70d75b78a35e4cf723be921ccf09657bec
Author: Filip Kastl <filip.kastl@gmail.com>
Date: Wed Sep 7 11:33:11 2022 +0200
added call to update_gimple_stmt ()
Diff:
---
gcc/sccp.cc | 29 +++++++++++++++++++++++------
1 file changed, 23 insertions(+), 6 deletions(-)
diff --git a/gcc/sccp.cc b/gcc/sccp.cc
index 32072c5d6d2..fd7f01db6b3 100644
--- a/gcc/sccp.cc
+++ b/gcc/sccp.cc
@@ -297,23 +297,40 @@ tarjan_compute_sccs (auto_vec<gphi *> &phis)
static void
replace_scc_by_value (vec<gphi *> scc, tree v)
{
+ // DEBUG
+ if (scc.length () >= 3)
+ {
+ std::cerr << "Replacing scc of size " << scc.length () << std::endl;
+ }
+
for (gphi *phi : scc)
{
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;
+ if (TREE_CODE (v) == SSA_NAME)
+ {
+ unsigned vnum_replaced_by = SSA_NAME_VERSION (v);
+ std::cerr << "Replacing " << vnum_get_replaced << " by " <<
+ vnum_replaced_by << std::endl;
+ }
+ else
+ {
+ std::cerr << "Replacing " << vnum_get_replaced << " by something"
+ << " that isn't an SSA name" << std::endl;
+ }
/* 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);
+ {
+ FOR_EACH_IMM_USE_ON_STMT (use_p, iter)
+ SET_USE (use_p, v);
+ update_stmt (use_stmt);
+ }
}
}
@@ -463,7 +480,7 @@ public:
unsigned
pass_sccp::execute (function *)
{
- //debug_phis (); // DEBUG
+ debug_phis (); // DEBUG
init_sccp ();
auto_vec<gphi *> phis = get_all_phis ();
More information about the Gcc-cvs
mailing list