diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d0e14d7b7b95..8bff6313c910 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 ffedb10f18fe..ac409f404aa9 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;