On Tuesday, May 27, 2014 07:48:58 PM Ian Romanick wrote:
> From: Ian Romanick <[email protected]>
> 
> 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 <[email protected]>

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 <[email protected]>

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

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to