[Mesa-dev] [PATCH excerpt] mesa: Wrap unlikely() around conditions leading to _mesa_error().
Brian Paul
brianp at vmware.com
Mon Mar 2 16:36:43 PST 2015
On 03/02/2015 03:48 PM, Kenneth Graunke wrote:
> Generated by the following Coccinelle semantic patch:
>
> @@
> expression E;
> @@
> if (
> -E
> +unlikely(E)
> )
> { ... _mesa_error(...) ... }
>
> Improves performance in OglBatch7 (a CPU-bound microbenchmark) on
> an i7 4750HQ (Haswell) by 1.08546% +/- 0.519734% (n=292).
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
I like it. But I have to say, when I first proposed doing this last
year (IIRC) someone poo-pood it. I don't remember who.
Acked-by: Brian Paul <brianp at vmware.com>
More information about the mesa-dev
mailing list