On Mon, 29 Jan 2007, Chris Li wrote:
> > > --- sparse.orig/expand.c 2007-01-16 11:13:18.000000000 -0800
> > > +++ sparse/expand.c 2007-01-16 11:13:28.000000000 -0800
> > > -#define SELECT_COST 20 /* Cut-off for turning a conditional
> > into a select */
> > > -#define BRANCH_COST 10 /* Cost of a conditional branch */
> > > +#define SELECT_COST 0 /* Cut-off for turning a conditional
> > into a select */
> > > +#define BRANCH_COST 0 /* Cost of a conditional branch */
> >
> > OK, I *think* that won't cause any harm.
>
> Linus, any words here? I have a vague impression that it has some thing to
> do with simplifying phi usage, but I am not sure what is the real side effect.
I just think the old costs are likely more realistic, but they were pulled
totally out of my *ss.
If it's more convenient to set them to zero, go wild. No deep meaning.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
