diff options
| author | Andrew Pinski <quic_apinski@quicinc.com> | 2024-10-10 04:44:23 +0000 |
|---|---|---|
| committer | Andrew Pinski <quic_apinski@quicinc.com> | 2024-10-10 15:32:36 +0000 |
| commit | dc3015ff0934a48176c43c0582d5a93029d298f9 (patch) | |
| tree | 4c58a042caca941ef848828c2ed05e76c4428a12 /gcc | |
| parent | fd8e590ff11266598d8f9b3d03d72ba7a6100512 (diff) | |
| download | gcc-dc3015ff0934a48176c43c0582d5a93029d298f9.zip gcc-dc3015ff0934a48176c43c0582d5a93029d298f9.tar.gz gcc-dc3015ff0934a48176c43c0582d5a93029d298f9.tar.bz2 | |
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_apinski@quicinc.com>
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/tree-ssa-phiopt.cc | 7 |
1 files changed, 1 insertions, 6 deletions
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; |