On Fri, Jul 10, 2009 at 11:33:59 +0100, Bill Crawford wrote: > Peter Hutterer wrote: > > > - nbytesRound = (nbytes + 3) & ~3; > > + nbytesRound = pad_to_dwords(nbytes); > > You're replacing two arithmetic operations with a function call here, > and then ... > > > - rep->length = npixels + (nbytesRound >> 2); > > + rep->length = npixels + num_dwords_for_bytes(nbytesRound); > > Is it really worth replacing a two-bit shift with a function call? > They're static inlines, so this shouldn't change anything. Did you check the generated code?
Cheers, Julien _______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
