[gcc r15-4241] phiopt: Remove candorest variable return instead

[Andrew Pinski via Gcc-cvs]

https://gcc.gnu.org/g:dc3015ff0934a48176c43c0582d5a93029d298f9

commit r15-4241-gdc3015ff0934a48176c43c0582d5a93029d298f9
Author: Andrew Pinski <quic_apin...@quicinc.com>
Date:   Thu Oct 10 04:44:23 2024 +0000

    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 <quic_apin...@quicinc.com>

Diff:
---
 gcc/tree-ssa-phiopt.cc | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/gcc/tree-ssa-phiopt.cc b/gcc/tree-ssa-phiopt.cc
index 43b65b362a39..f3ee3a80c0f8 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;