[Mesa-dev] [PATCH 1/8] meta: Put _mesa_meta_in_progress in the header file

Kenneth Graunke kenneth at whitecape.org
Sun Dec 21 23:35:57 PST 2014


On Friday, December 19, 2014 02:20:52 PM Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> 
> ...so that it can be inlined in the two places that call it.
> 
> On Bay Trail-D using Fedora 20 compile flags (-m64 -O2 -mtune=generic
> for 64-bit and -m32 -march=i686 -mtune=atom for 32-bit), affects
> Gl32Batch7:
> 
> 32-bit: No difference proven at 95.0% confidence (n=120)
> 64-bit: Difference at 95.0% confidence 1.24042% +/- 0.382277% (n=40)
> 
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/mesa/drivers/common/meta.c | 10 ----------
>  src/mesa/drivers/common/meta.h |  7 +++++--
>  2 files changed, 5 insertions(+), 12 deletions(-)

I sent comments on patches 2, 3, 4, and 8.

The series is:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141221/179d460e/attachment.sig>


More information about the mesa-dev mailing list