Vulkan: Don't dirty index/vertex buffers on each syncState.

This will help for the command graph refactor so we can set up the command buffer barriers before we start the render pass for draw. Bug: angleproject:3539 Bug: angleproject:4029 Change-Id: If5a55db6ac1e06609a3a1e2245d7ab351152886a Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2028450Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Tim Van Patten <timvp@google.com> Commit-Queue: Jamie Madill <jmadill@chromium.org>

Vulkan: Don't dirty index/vertex buffers on each syncState.
d9884d3e · Jamie Madill · Commit Bot · ee07cb31 · d9884d3e
Commit d9884d3e authored Jan 29, 2020 by Jamie Madill Committed by Commit Bot Jan 30, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

ContextVk.cpp src/libANGLE/renderer/vulkan/ContextVk.cpp +2 -1

No files found.
--- a/src/libANGLE/renderer/vulkan/ContextVk.cpp
+++ b/src/libANGLE/renderer/vulkan/ContextVk.cpp
@@ -2309,7 +2309,7 @@ angle::Result ContextVk::syncState(const gl::Context *context,
    if ((dirtyBits & mPipelineDirtyBitsMask).any() &&
        (glState.getProgram() == nullptr || !glState.getProgram()->isCompute()))
    {
-        invalidateVertexAndIndexBuffers();
+        invalidateCurrentGraphicsPipeline();
    }

    for (auto iter = dirtyBits.begin(), endIter = dirtyBits.end(); iter != endIter; ++iter)
@@ -2552,6 +2552,7 @@ angle::Result ContextVk::syncState(const gl::Context *context,
                    // later.
                    invalidateDefaultAttributes(
                        context->getStateCache().getActiveDefaultAttribsMask());
+                    invalidateVertexAndIndexBuffers();
                    bool useVertexBuffer = (mProgram->getState().getMaxActiveAttribLocation() > 0);
                    mNonIndexedDirtyBitsMask.set(DIRTY_BIT_VERTEX_BUFFERS, useVertexBuffer);
                    mIndexedDirtyBitsMask.set(DIRTY_BIT_VERTEX_BUFFERS, useVertexBuffer);