On Wed, 2007-06-06 10:15:41 -0700, Brian Sidebotham <[EMAIL PROTECTED]> wrote:
> --- ./gcc/value-prof.c Mon Jun 4 17:27:14 2007
> +++ ./gcc/value-prof.c Wed Jun 6 10:13:20 2007
>
> @@ -883,15 +883,19 @@ tree_mod_subtract (tree stmt, tree opera
> e12 = split_block (bb, bb1end);
> bb2 = e12->dest;
> bb2->count = all - count1;
> -
> +
> if (ncounts) /* Assumed to be 0 or 1. */
> {
> e23 = split_block (bb2, bb2end);
> bb3 = e23->dest;
> bb3->count = all - count1 - count2;
> + e34 = split_block (bb3 : bb2, bb3end);
`---------'
First of all, you should compile GCC with your patch applied, and then
run the testsuite. No new regressions should occur (or you should have
a good reason for them!)
> + }
> + else
> + {
> + e34 = split_block (bb2, bb3end);
> }
>
> - e34 = split_block (ncounts ? bb3 : bb2, bb3end);
> bb4 = e34->dest;
> bb4->count = all;
However, this won't even compile, so please redo it, compile it, test
it with the regression suite and submit it with a ChangeLog entry.
(Just plain text, don't prepare it as a patch to the ChangeLog file!)
MfG, JBG
--
Jan-Benedict Glaw [EMAIL PROTECTED] +49-172-7608481
Signature of: Gib Dein Bestes. Dann übertriff Dich selbst!
the second :
signature.asc
Description: Digital signature
