From c0f9d1fd7e54434e4f06b20b9a61a9d35c578223 Mon Sep 17 00:00:00 2001 From: Richard Biener Date: Fri, 3 May 2019 07:05:34 +0000 Subject: [PATCH] passes.c (execute_function_todo): Remove dead code. 2019-05-03 Richard Biener * passes.c (execute_function_todo): Remove dead code. From-SVN: r270832 --- gcc/ChangeLog | 4 ++++ gcc/passes.c | 24 +++++------------------- 2 files changed, 9 insertions(+), 19 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 53428649c37..65df01190cc 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2019-05-03 Richard Biener + + * passes.c (execute_function_todo): Remove dead code. + 2019-05-02 Segher Boessenkool * config/rs6000/rs6000.c (rs6000_dbx_register_number): Do not use diff --git a/gcc/passes.c b/gcc/passes.c index 02561f8bf18..a9c22889d71 100644 --- a/gcc/passes.c +++ b/gcc/passes.c @@ -1924,26 +1924,12 @@ execute_function_todo (function *fn, void *data) push_cfun (fn); - /* Always cleanup the CFG before trying to update SSA. */ + /* If we need to cleanup the CFG let it perform a needed SSA update. */ if (flags & TODO_cleanup_cfg) - { - cleanup_tree_cfg (flags & TODO_update_ssa_any); - - /* When cleanup_tree_cfg merges consecutive blocks, it may - perform some simplistic propagation when removing single - valued PHI nodes. This propagation may, in turn, cause the - SSA form to become out-of-date (see PR 22037). So, even - if the parent pass had not scheduled an SSA update, we may - still need to do one. */ - if (!(flags & TODO_update_ssa_any) && need_ssa_update_p (cfun)) - flags |= TODO_update_ssa; - } - - if (flags & TODO_update_ssa_any) - { - unsigned update_flags = flags & TODO_update_ssa_any; - update_ssa (update_flags); - } + cleanup_tree_cfg (flags & TODO_update_ssa_any); + else if (flags & TODO_update_ssa_any) + update_ssa (flags & TODO_update_ssa_any); + gcc_assert (!need_ssa_update_p (fn)); if (flag_tree_pta && (flags & TODO_rebuild_alias)) compute_may_aliases (); -- 2.30.2