commit dd34b3b9b707c93b3323ac857793ddeae5f7a777
author Jamie Madill <jmadill@chromium.org> Wed Jan 16 09:59:54 2019 -0500
committer Commit Bot <commit-bot@chromium.org> Wed Jan 16 18:25:16 2019 +0000
tree c563fbad6f1b741adac67e9a4c8329cbd7f0c4ed
parent a5d2d55d2a27987b893e1b37f320a3c8eeaae4d3

Pack VertexAttribType enum.

This improves performance slightly in vertex array format checks.
Instead of needing to switch on GLenum values we can use packed arrays
and tables to determine the values we need.

Does not significantly affect performance but will enable future work.

Bug: angleproject:3074
Change-Id: I6f4821a463e9b41fe3f8c8967eb3ed4c1d6b84be
Reviewed-on: https://chromium-review.googlesource.com/c/1393903
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>

src/libANGLE/Context.cpp

diff --git a/src/libANGLE/Context.cpp b/src/libANGLE/Context.cpp
index 0c93d95..545a0ac 100644
--- a/src/libANGLE/Context.cpp
+++ b/src/libANGLE/Context.cpp
@@ -4781,7 +4781,7 @@
 
 void Context::vertexAttribPointer(GLuint index,
                                   GLint size,
-                                  GLenum type,
+                                  VertexAttribType type,
                                   GLboolean normalized,
                                   GLsizei stride,
                                   const void *ptr)
@@ -4793,7 +4793,7 @@
 
 void Context::vertexAttribFormat(GLuint attribIndex,
                                  GLint size,
-                                 GLenum type,
+                                 VertexAttribType type,
                                  GLboolean normalized,
                                  GLuint relativeOffset)
 {
@@ -4804,7 +4804,7 @@
 
 void Context::vertexAttribIFormat(GLuint attribIndex,
                                   GLint size,
-                                  GLenum type,
+                                  VertexAttribType type,
                                   GLuint relativeOffset)
 {
     mState.setVertexAttribFormat(attribIndex, size, type, false, true, relativeOffset);
@@ -4830,7 +4830,7 @@
 
 void Context::vertexAttribIPointer(GLuint index,
                                    GLint size,
-                                   GLenum type,
+                                   VertexAttribType type,
                                    GLsizei stride,
                                    const void *pointer)
 {