From dc3015ff0934a48176c43c0582d5a93029d298f9 Mon Sep 17 00:00:00 2001 From: Andrew Pinski Date: Thu, 10 Oct 2024 04:44:23 +0000 Subject: phiopt: Remove candorest variable return instead After r15-3560-gb081e6c860eb9688d24365d39, the setting of candorest with the break can just change to a return since this is inside a lambda now. Bootstrapped and tested on x86_64-linux-gnu. gcc/ChangeLog: * tree-ssa-phiopt.cc (pass_phiopt::execute): Remove candorest and return instead of setting candorest. Signed-off-by: Andrew Pinski --- gcc/tree-ssa-phiopt.cc | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'gcc/tree-ssa-phiopt.cc') diff --git a/gcc/tree-ssa-phiopt.cc b/gcc/tree-ssa-phiopt.cc index 43b65b3..f3ee3a8 100644 --- a/gcc/tree-ssa-phiopt.cc +++ b/gcc/tree-ssa-phiopt.cc @@ -4322,7 +4322,6 @@ pass_phiopt::execute (function *) } gimple_stmt_iterator gsi; - bool candorest = true; /* Check that we're looking for nested phis. */ basic_block merge = diamond_p ? EDGE_SUCC (bb2, 0)->dest : bb2; @@ -4338,15 +4337,11 @@ pass_phiopt::execute (function *) tree arg1 = gimple_phi_arg_def (phi, e2->dest_idx); if (value_replacement (bb, bb1, e1, e2, phi, arg0, arg1) == 2) { - candorest = false; cfgchanged = true; - break; + return; } } - if (!candorest) - return; - gphi *phi = single_non_singleton_phi_for_edges (phis, e1, e2); if (!phi) return; -- cgit v1.1