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
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to