[Mesa-dev] [PATCH 02/70] i965: Pack read-only booleans into a bitfield

Chris Wilson chris at chris-wilson.co.uk
Mon Aug 10 02:59:07 PDT 2015


On Fri, Aug 07, 2015 at 09:13:06PM +0100, Chris Wilson wrote:
> GCC's read access for single bits in a bitfield is reasonable (just a
> move + flag comparison), so let's save some cachelines by packing the
> write-once/read-many booleans together.
> 
>    text	   data	    bss	    dec	    hex	filename
> 6490134	 191992	  26192	6708318	 665c5e	lib64/i965_dri.so
> 6491766	 191992	  26192	6709950	 6662be	lib64/i965_dri.so
> 
> Small inflation due to the extra immediate masks and entirely dubious as
> to whether it is worth it.

A run though Martin's testsuite on Braswell also suggests the extra code
hurts noticeably, so just ignore this patch...
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the mesa-dev mailing list