diff options
author | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2017-10-30 19:37:39 +0100 |
---|---|---|
committer | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2017-10-31 10:16:38 +0100 |
commit | 5010436e09f3a7acae679e192d2fe65f04c87c79 (patch) | |
tree | 0075216a915d0831c0104b4e34647e0f739eb4a2 | |
parent | 11fdc2cd3420aa1541cc4c3c733e9a878308eb55 (diff) |
radv: bail out when binding the same vertex buffers
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
-rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 00ed7182a7d..4b38ece10fe 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2259,14 +2259,28 @@ void radv_CmdBindVertexBuffers( { RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer); struct radv_vertex_binding *vb = cmd_buffer->state.vertex_bindings; + bool changed = false; /* We have to defer setting up vertex buffer since we need the buffer * stride from the pipeline. */ assert(firstBinding + bindingCount <= MAX_VBS); for (uint32_t i = 0; i < bindingCount; i++) { - vb[firstBinding + i].buffer = radv_buffer_from_handle(pBuffers[i]); - vb[firstBinding + i].offset = pOffsets[i]; + uint32_t idx = firstBinding + i; + + if (!changed && + (vb[idx].buffer != radv_buffer_from_handle(pBuffers[i]) || + vb[idx].offset != pOffsets[i])) { + changed = true; + } + + vb[idx].buffer = radv_buffer_from_handle(pBuffers[i]); + vb[idx].offset = pOffsets[i]; + } + + if (!changed) { + /* No state changes. */ + return; } cmd_buffer->state.vb_dirty = true; |