From ae5512dd4fbfb031cc967fc154a534863d4c05f5 Mon Sep 17 00:00:00 2001 From: Jan Hubicka Date: Thu, 8 Jun 2017 16:47:26 +0200 Subject: [PATCH] opts.c (finish_options): x_flag_reorder_blocks_and_partition no longer requires x_flag_profile_use. * opts.c (finish_options): x_flag_reorder_blocks_and_partition no longer requires x_flag_profile_use. From-SVN: r249019 --- gcc/ChangeLog | 5 +++++ gcc/opts.c | 10 ---------- 2 files changed, 5 insertions(+), 10 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d0e14d7b7b9..8bff6313c91 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2017-06-08 Jan Hubicka + + * opts.c (finish_options): x_flag_reorder_blocks_and_partition no + longer requires x_flag_profile_use. + 2017-06-08 Jan Hubicka * cfgrtl.c (cfg_layout_initialize): Check crtl->has_bb_partition diff --git a/gcc/opts.c b/gcc/opts.c index ffedb10f18f..ac409f404aa 100644 --- a/gcc/opts.c +++ b/gcc/opts.c @@ -863,16 +863,6 @@ finish_options (struct gcc_options *opts, struct gcc_options *opts_set, opts->x_flag_reorder_blocks = 1; } - /* Disable -freorder-blocks-and-partition when -fprofile-use is not in - effect. Function splitting was not actually being performed in that case, - as probably_never_executed_bb_p does not distinguish any basic blocks as - being cold vs hot when there is no profile data. Leaving it enabled, - however, causes the assembly code generator to create (empty) cold - sections and labels, leading to unnecessary size overhead. */ - if (opts->x_flag_reorder_blocks_and_partition - && !opts_set->x_flag_profile_use) - opts->x_flag_reorder_blocks_and_partition = 0; - if (opts->x_flag_reorder_blocks_and_partition && !opts_set->x_flag_reorder_functions) opts->x_flag_reorder_functions = 1; -- 2.30.2