Re: [Mesa-dev] [PATCH 3/3] mesa: Clean up gl_array_object::_MaxElement computation.

2012-01-11 Thread Mathias Fröhlich
Hi Eric, Thanks for the review. I will send shortly an updated patch. ... except: On Friday, December 30, 2011 12:13:36 Eric Anholt wrote: > Also, I think that helper would be easier to read as; > > + GLbitfield64 enabled = arrayObj->_Enabled; > + > + if (enabled & VERT_ATTRIB_GENERIC0) >

Re: [Mesa-dev] [PATCH 3/3] mesa: Clean up gl_array_object::_MaxElement computation.

2011-12-30 Thread Eric Anholt
On Thu, 29 Dec 2011 23:14:25 +0100, Mathias Fröhlich wrote: > Use a bitmask approach to compute gl_array_object::_MaxElement. > To make this work reliably depending on the shader type actually used, > make use of the newly introduced typed bitmaks getters. > With this change I gain about 5% draw

[Mesa-dev] [PATCH 3/3] mesa: Clean up gl_array_object::_MaxElement computation.

2011-12-29 Thread Mathias Fröhlich
Use a bitmask approach to compute gl_array_object::_MaxElement. To make this work reliably depending on the shader type actually used, make use of the newly introduced typed bitmaks getters. With this change I gain about 5% draw time on some osgviewer examples. Signed-off-by: Mathias Fröhlich ---