diff options
author | Richard Biener <rguenther@suse.de> | 2017-04-12 09:41:02 +0000 |
---|---|---|
committer | Richard Biener <rguenth@gcc.gnu.org> | 2017-04-12 09:41:02 +0000 |
commit | 7f7e9dae36c8dd57ab89dae47a87903bbfefb530 (patch) | |
tree | bcf8e9fbdfc31a7ebdbb48a33d5e1ac9a7ec130d | |
parent | de81c2f2f9499ba85257b597cdfce3f088758d48 (diff) |
re PR tree-optimization/79390 (10% performance drop in SciMark2 LU after r242550)
2017-04-12 Richard Biener <rguenther@suse.de>
PR tree-optimization/79390
* gimple-ssa-split-paths.c (is_feasible_trace): Restrict
threading case even more.
From-SVN: r246869
-rw-r--r-- | gcc/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/gimple-ssa-split-paths.c | 10 |
2 files changed, 13 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 1458921cb44..d59a0e997ee 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-04-12 Richard Biener <rguenther@suse.de> + + PR tree-optimization/79390 + * gimple-ssa-split-paths.c (is_feasible_trace): Restrict + threading case even more. + 2017-04-12 Segher Boessenkool <segher@kernel.crashing.org> PR target/80382 diff --git a/gcc/gimple-ssa-split-paths.c b/gcc/gimple-ssa-split-paths.c index df88c76aa53..70651c5dea3 100644 --- a/gcc/gimple-ssa-split-paths.c +++ b/gcc/gimple-ssa-split-paths.c @@ -249,13 +249,17 @@ is_feasible_trace (basic_block bb) imm_use_iterator iter2; FOR_EACH_IMM_USE_FAST (use2_p, iter2, gimple_phi_result (stmt)) { - if (is_gimple_debug (USE_STMT (use2_p))) + gimple *use_stmt = USE_STMT (use2_p); + if (is_gimple_debug (use_stmt)) continue; - basic_block use_bb = gimple_bb (USE_STMT (use2_p)); + basic_block use_bb = gimple_bb (use_stmt); if (use_bb != bb && dominated_by_p (CDI_DOMINATORS, bb, use_bb)) { - found_useful_phi = true; + if (gcond *cond = dyn_cast <gcond *> (use_stmt)) + if (gimple_cond_code (cond) == EQ_EXPR + || gimple_cond_code (cond) == NE_EXPR) + found_useful_phi = true; break; } } |