On Tuesday, May 27, 2014 07:48:58 PM Ian Romanick wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
> 
> Just use ir_variable::data.binding... because that's the where the
> binding is stored for everything else that can use layout(binding=).
> 
> No change in the peak ir_variable memory usage in a trimmed apitrace of
> dota2 on 64-bit.
> 
> Reduces the peak ir_variable memory usage in a trimmed apitrace of dota2
> by 102KiB on 32-bit.
> 
> Before: IR MEM: variable usage / name / total: 4955496 915817 5871313
> After:  IR MEM: variable usage / name / total: 4850844 915817 5766661
> 
> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>

This patch seems good to me - it puts the value of the binding layout 
qualifier in the same place for everything that uses it.

Patches 3-4 are:
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to