http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46916

--- Comment #43 from Jack Howarth <howarth at nitro dot med.uc.edu> 2010-12-14 
23:21:35 UTC ---
(In reply to comment #39)
> The patch I had in mind
> http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01129.html
> 
> The hunk
> Index: gcc/opts.c
> ===================================================================
> --- gcc/opts.c    (revision 167793)
> +++ gcc/opts.c    (working copy)
> @@ -723,6 +723,9 @@ finish_options (struct gcc_options *opts, struct g
>        opts->x_flag_reorder_blocks = 1;
>      }
> 
> +  if (opts->x_flag_reorder_blocks_and_partition)
> +    opts->x_flag_reorder_blocks = 1;
> +
>    /* If user requested unwind info, then turn off the partitioning
>       optimization.  */
> 
> should be
> Index: gcc/opts.c
> ===================================================================
> --- gcc/opts.c    (revision 167793)
> +++ gcc/opts.c    (working copy)
> @@ -723,6 +723,9 @@ finish_options (struct gcc_options *opts, struct g
>        opts->x_flag_reorder_blocks = 1;
>      }
> 
> +  if (opts->x_flag_reorder_blocks_and_partition)
> +    opts->x_flag_reorder_functions = 1;
> +
>    /* If user requested unwind info, then turn off the partitioning
>       optimization.  */
> 
> sorry for typo.

The cited patch doesn't have any such change in that particular context. Was
this entire hunk missing from the posted patch?

Reply via email to