opts.c (finish_options): x_flag_reorder_blocks_and_partition no longer requires x_fla...

	* opts.c (finish_options): x_flag_reorder_blocks_and_partition no
	longer requires x_flag_profile_use.

From-SVN: r249019

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d0e14d7b7b9569f70953c7406bc97a8961073b45..8bff6313c910d5201089da55ce6464a15340247d 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+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

diff --git a/gcc/opts.c b/gcc/opts.c
index ffedb10f18fee785a4af4267cc17ec4ad27ddfe7..ac409f404aa9319f3b3db550192c86d4329aa6d0 100644 (file)
--- 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;