]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
phiopt: Remove candorest variable return instead
authorAndrew Pinski <quic_apinski@quicinc.com>
Thu, 10 Oct 2024 04:44:23 +0000 (04:44 +0000)
committerAndrew Pinski <quic_apinski@quicinc.com>
Thu, 10 Oct 2024 15:32:36 +0000 (15:32 +0000)
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_apinski@quicinc.com>
gcc/tree-ssa-phiopt.cc

index 43b65b362a3964f2aa9905d087871c5f93cb25ec..f3ee3a80c0f8be9dd87c13801e45b4e16dcaca8d 100644 (file)
@@ -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;