mesa: shrink gl_vertex_array
authorBrian Paul <brianp@vmware.com>
Fri, 26 Jan 2018 21:49:41 +0000 (14:49 -0700)
committerBrian Paul <brianp@vmware.com>
Tue, 30 Jan 2018 04:15:52 +0000 (21:15 -0700)
Inspired by Marek's earlier patch, but goes a little further.
Sort fields from largest to smallest.  Use bitfields.

Reduced from 48 bytes to 32.  Also reduces size of gl_vertex_array_object
from 4144 to 3632

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/mesa/main/mtypes.h

index b0eeeafbcdafc4fbeb9b4e080b441d0230d24d68..78132de425450856b93138426e07b6ab6f78ddf6 100644 (file)
@@ -1497,18 +1497,19 @@ struct gl_pixelstore_attrib
  */
 struct gl_vertex_array
 {
-   GLint Size;                  /**< components per element (1,2,3,4) */
-   GLenum16 Type;               /**< datatype: GL_FLOAT, GL_INT, etc */
-   GLenum16 Format;             /**< default: GL_RGBA, but may be GL_BGRA */
+   /** if NULL, vertex data are in user memory */
+   struct gl_buffer_object *BufferObj;
+   /** Pointer into user memory, or offset into the BufferObj */
+   const GLubyte *Ptr;
    GLsizei StrideB;            /**< actual stride in bytes */
-   GLuint _ElementSize;         /**< size of each element in bytes */
-   const GLubyte *Ptr;          /**< Points to array data */
-   GLboolean Normalized;        /**< GL_ARB_vertex_program */
-   GLboolean Integer;           /**< Integer-valued? */
-   GLboolean Doubles;       /**< double precision values are not converted to floats */
    GLuint InstanceDivisor;      /**< GL_ARB_instanced_arrays */
-
-   struct gl_buffer_object *BufferObj;/**< GL_ARB_vertex_buffer_object */
+   GLenum16 Type;               /**< datatype: GL_FLOAT, GL_INT, etc */
+   GLenum16 Format;             /**< default: GL_RGBA, but may be GL_BGRA */
+   unsigned Size:4;             /**< components per element (1,2,3,4) */
+   unsigned _ElementSize:8;     /**< in bytes, up to 4*sizeof(GLdouble) */
+   unsigned Normalized:1;       /**< GL_ARB_vertex_program */
+   unsigned Integer:1;          /**< Integer-valued? */
+   unsigned Doubles:1;          /**< doubles not converted to floats */
 };