aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
Diffstat (limited to 'gcc')
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/opts.c10
2 files changed, 5 insertions, 10 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d0e14d7..8bff631 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,10 @@
2017-06-08 Jan Hubicka <hubicka@ucw.cz>
+ * opts.c (finish_options): x_flag_reorder_blocks_and_partition no
+ longer requires x_flag_profile_use.
+
+2017-06-08 Jan Hubicka <hubicka@ucw.cz>
+
* cfgrtl.c (cfg_layout_initialize): Check crtl->has_bb_partition
instead of flag_reorder_blocks_and_partition.
* dbxout.c (dbxout_function_end): Likewise.
diff --git a/gcc/opts.c b/gcc/opts.c
index ffedb10..ac409f4 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;