This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA} Patch to implement "Switch statement fdo reordering"
- From: Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>
- To: Edmar Wienskoski-RA8797 <edmar at freescale dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 29 Apr 2008 19:48:56 +0200
- Subject: Re: [RFA} Patch to implement "Switch statement fdo reordering"
- References: <481742FC.9020308@freescale.com>
Hello Edmar,
a couple of typo nits:
* Edmar Wienskoski-RA8797 wrote on Tue, Apr 29, 2008 at 05:47:08PM CEST:
[...]
> --- gcc/gcc/doc/invoke.texi (revision 134561)
> +++ gcc/gcc/doc/invoke.texi (working copy)
> @@ -6348,7 +6348,8 @@
> optimization. You must use @option{-fprofile-generate} both when
> compiling and when linking your program.
>
> -The following options are enabled: @code{-fprofile-arcs}, @code{-fprofile-values}, @code{-fvpt}.
> +The following options are enabled: @code{-fprofile-arcs}, @code{-fprofile-values},
> +@code{-fvpt}, @code{-fvpt-case-rebalance}.
>
> If @var{path} is specified, GCC will look at the @var{path} to find
> the profile feeedback data files. See @option{-fprofile-dir}.
I realize this typo was here before, but please
s/feeedback/feedback/
> --- gcc/gcc/stmt.c (revision 134561)
> +++ gcc/gcc/stmt.c (working copy)
> @@ -2453,6 +2459,222 @@
[...]
> + /* Distribute the cost of bigs and negatives uniformely */
s/uniformely/uniformly/
> + for (cn = case_list; cn; cn = cn->right)
> + {
> + if (tree_int_cst_equal (cn->low, cn->high))
> + {
[...]
Cheers,
Ralf