diff options
| author | Richard Biener <rguenther@suse.de> | 2020-04-02 10:46:20 +0200 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2020-04-02 16:53:21 +0200 |
| commit | 75efe9cb1f8938a713ce540dc3b27bc2afcd3fae (patch) | |
| tree | 69f0c6905f576eeedd84101d3bfb66a72a2c4331 /gcc/ipa-icf-gimple.c | |
| parent | 54af95767e887d63dc332731738e642536d87a48 (diff) | |
c/94392 - only enable -ffinite-loops for C++
This does away with enabling -ffinite-loops at -O2+ for all languages
and instead enables it selectively for C++ only.
It also makes -ffinite-loops loop-private at CFG construction time
fixing correctness issues with inlining.
2020-04-02 Richard Biener <rguenther@suse.de>
PR c/94392
* c-opts.c (c_common_post_options): Enable -ffinite-loops
for -O2 and C++11 or newer.
* common.opt (ffinite-loops): Initialize to zero.
* opts.c (default_options_table): Remove OPT_ffinite_loops
entry.
* cfgloop.h (loop::finite_p): New member.
* cfgloopmanip.c (copy_loop_info): Copy finite_p.
* ipa-icf-gimple.c (func_checker::compare_loops): Compare
finite_p.
* lto-streamer-in.c (input_cfg): Stream finite_p.
* lto-streamer-out.c (output_cfg): Likewise.
* tree-cfg.c (replace_loop_annotate): Initialize finite_p
from flag_finite_loops at CFG build time.
* tree-ssa-loop-niter.c (finite_loop_p): Check the loops
finite_p flag instead of flag_finite_loops.
* doc/invoke.texi (ffinite-loops): Adjust documentation of
default setting.
* gcc.dg/torture/pr94392.c: New testcase.
Diffstat (limited to 'gcc/ipa-icf-gimple.c')
| -rw-r--r-- | gcc/ipa-icf-gimple.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/gcc/ipa-icf-gimple.c b/gcc/ipa-icf-gimple.c index 3e5b2d4bd6d..d306fec56ce 100644 --- a/gcc/ipa-icf-gimple.c +++ b/gcc/ipa-icf-gimple.c @@ -395,6 +395,8 @@ func_checker::compare_loops (basic_block bb1, basic_block bb2) return return_false_with_msg ("dont_vectorize"); if (l1->force_vectorize != l2->force_vectorize) return return_false_with_msg ("force_vectorize"); + if (l1->finite_p != l2->finite_p) + return return_false_with_msg ("finite_p"); if (l1->unroll != l2->unroll) return return_false_with_msg ("unroll"); if (!compare_variable_decl (l1->simduid, l2->simduid)) |