diff options
| author | Richard Biener <rguenther@suse.de> | 2022-01-12 15:25:07 +0100 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2022-01-12 16:18:20 +0100 |
| commit | 2f62294dec1f3af59dd7505c058b0af38c2d1524 (patch) | |
| tree | aab37aab5c09a20b6c7444090899ef3f13c171c8 /gcc/tree-ssa-pre.c | |
| parent | 2c16dfe6268eeeb4b7924ff423e274fa00894a4d (diff) | |
| download | gcc-2f62294dec1f3af59dd7505c058b0af38c2d1524.zip gcc-2f62294dec1f3af59dd7505c058b0af38c2d1524.tar.gz gcc-2f62294dec1f3af59dd7505c058b0af38c2d1524.tar.bz2 | |
tree-optimization/103990 - fix CFG cleanup regression from PRE change
This adjusts the CFG cleanup flow back to what it was before the
last change which fixes the observed regression of 541.leela_r with
LTO and FDO.
2022-01-12 Richard Biener <rguenther@suse.de>
PR tree-optimization/103990
* tree-pass.h (tail_merge_optimize): Drop unused argument.
* tree-ssa-tail-merge.c (tail_merge_optimize): Likewise.
* tree-ssa-pre.c (pass_pre::execute): Retain TODO_cleanup_cfg
and adjust call to tail_merge_optimize.
Diffstat (limited to 'gcc/tree-ssa-pre.c')
| -rw-r--r-- | gcc/tree-ssa-pre.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/gcc/tree-ssa-pre.c b/gcc/tree-ssa-pre.c index ab24fa9..5113256 100644 --- a/gcc/tree-ssa-pre.c +++ b/gcc/tree-ssa-pre.c @@ -4442,7 +4442,6 @@ pass_pre::execute (function *fun) if (todo & TODO_cleanup_cfg) { cleanup_tree_cfg (); - todo &= ~TODO_cleanup_cfg; need_crit_edge_split = true; } @@ -4458,9 +4457,8 @@ pass_pre::execute (function *fun) It should either: - call merge_blocks after each tail merge iteration - call merge_blocks after all tail merge iterations - - mark TODO_cleanup_cfg when necessary - - share the cfg cleanup with fini_pre. */ - todo |= tail_merge_optimize (todo, need_crit_edge_split); + - mark TODO_cleanup_cfg when necessary. */ + todo |= tail_merge_optimize (need_crit_edge_split); free_rpo_vn (); |