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>
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