[PATCH 2/7] PowerPC: add unlikely() to BUG_ON()
i at coly.li
Fri Jan 28 22:02:19 EST 2011
On 2011年01月28日 18:14, Andreas Schwab Wrote:
> "David Laight"<David.Laight at ACULAB.COM> writes:
>> Also, as (I think) in some of the generated code quoted,
>> use of __builtin_expect() with a boolean expression can
>> force some versions of gcc to generate the integer
>> value of the expression
> That's more likely a side effect of the definition of likely/unlikely:
> they expand to !!(x).
It seems whether or not using unlikely() inside arch implemented BUG_ON() is arch dependent. Maybe a reasonable method
to use BUG_ON() is,
1) do not explicitly use unlikely() when using macro BUG_ON().
2) whether or not using unlikely() inside BUG_ON(), it depends on the implementation of BUG_ON(), including arch
So from current feed back, doing "unlikely() optimization" here doesn't make anything better.
Thanks for all of your feed back :-)
More information about the Linuxppc-dev