Now after r17-1753-g2e5184c4a93c4b0d4067, all parts of
cs-elim manually keeps the VOPS up todate and don't need
any renaming, don't return TODO_TODO_update_ssa_only_virtuals
when something is inserted.

Pushed as obvious after a bootstrap/test on x86_64-linux-gnu.

gcc/ChangeLog:

        * tree-ssa-phiopt.cc (pass_cselim::execute): Don't
        return TODO_TODO_update_ssa_only_virtuals.

Signed-off-by: Andrew Pinski <[email protected]>
---
 gcc/tree-ssa-phiopt.cc | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/gcc/tree-ssa-phiopt.cc b/gcc/tree-ssa-phiopt.cc
index 7a623fc4c0c..5c263df70eb 100644
--- a/gcc/tree-ssa-phiopt.cc
+++ b/gcc/tree-ssa-phiopt.cc
@@ -4643,10 +4643,8 @@ pass_cselim::execute (function *)
   /* If the CFG has changed, we should cleanup the CFG.  */
   if (cfgchanged)
     {
-      /* In cond-store replacement we have added some loads on edges
-         and new VOPS (as we moved the store, and created a load).  */
       gsi_commit_edge_inserts ();
-      todo = TODO_cleanup_cfg | TODO_update_ssa_only_virtuals;
+      todo = TODO_cleanup_cfg;
     }
   scev_finalize ();
   loop_optimizer_finalize ();
-- 
2.43.0

Reply via email to