cfgcleanup: merge tree_forwarder_block_p and remove_forwarder_block into maybe_remove_forwarder_block

remove_forwarder_block is never called without calling tree_forwarder_block_p
before hand so this merges the two functions and simplifies the code slightly.
tree_forwarder_block_p was only returning true at the very end (except for one
location in the code checking the loop which is fixed here) so this nature to
merge the two functions even.

move_debug_stmts_from_forwarder and phi_alternatives_equal definitions were moved
to before maybe_remove_forwarder_block.

Bootstrapped and tested on x86_64-linux-gnu.

gcc/ChangeLog:

	* tree-cfgcleanup.cc (tree_forwarder_block_p): Merge this and ...
	(remove_forwarder_block): This into ...
	(maybe_remove_forwarder_block): Here.
	(cleanup_tree_cfg_bb): Call only maybe_remove_forwarder_block.
	(pass_merge_phi::execute): Likewise.

Signed-off-by: Andrew Pinski <andrew.pinski@oss.qualcomm.com>

0 files changed, 0 insertions, 0 deletions