commit 6e18a23825027d5e62073f0357ad48024a583ce4
author Jamie Madill <jmadill@chromium.org> Wed Jan 16 13:27:14 2019 -0500
committer Commit Bot <commit-bot@chromium.org> Wed Jan 16 20:01:14 2019 +0000
tree 53b6d08f036a1f513ae3ee34e49d96c481087a32
parent dd34b3b9b707c93b3323ac857793ddeae5f7a777

Optimize more front-end VertexArray binding.

Improves perf slightly (1-2%) in the Vulkan VBO state change test.

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

src/libANGLE/Context.cpp

diff --git a/src/libANGLE/Context.cpp b/src/libANGLE/Context.cpp
index 545a0ac..a4ae976 100644
--- a/src/libANGLE/Context.cpp
+++ b/src/libANGLE/Context.cpp
@@ -4787,7 +4787,7 @@
                                   const void *ptr)
 {
     mState.setVertexAttribPointer(this, index, mState.getTargetBuffer(BufferBinding::Array), size,
-                                  type, ConvertToBool(normalized), false, stride, ptr);
+                                  type, ConvertToBool(normalized), stride, ptr);
     mStateCache.onVertexArrayStateChange(this);
 }
 
@@ -4834,8 +4834,8 @@
                                    GLsizei stride,
                                    const void *pointer)
 {
-    mState.setVertexAttribPointer(this, index, mState.getTargetBuffer(BufferBinding::Array), size,
-                                  type, false, true, stride, pointer);
+    mState.setVertexAttribIPointer(this, index, mState.getTargetBuffer(BufferBinding::Array), size,
+                                   type, stride, pointer);
     mStateCache.onVertexArrayStateChange(this);
 }