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

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;