aboutsummaryrefslogtreecommitdiff
path: root/layers
diff options
context:
space:
mode:
authorTobin Ehlis <tobine@google.com>2017-09-28 12:46:58 -0600
committerTobin Ehlis <tobine@google.com>2017-09-28 13:26:43 -0700
commit0bdbf3c2d36a556ab2aa9ccf27b62a6ef5d21016 (patch)
tree8b468d186408aec6019c04a8fbe0eeca398d9670 /layers
parentd6a76d89728d9364decb0b45c30416dc0b3062c1 (diff)
downloadusermoji-0bdbf3c2d36a556ab2aa9ccf27b62a6ef5d21016.tar.xz
layers:Remove invalid renderPass binding
There's no reason to create a binding between RP used in pipeline creation and the cmd buffer that pipeline is bound to. This can lead to a bug where CB is incorrectly invalidated if that RP is destroyed after pipeline is bound to CB.
Diffstat (limited to 'layers')
-rw-r--r--layers/core_validation.cpp8
1 files changed, 0 insertions, 8 deletions
diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp
index 31e090f0..9671e5f7 100644
--- a/layers/core_validation.cpp
+++ b/layers/core_validation.cpp
@@ -5198,14 +5198,6 @@ VKAPI_ATTR void VKAPI_CALL CmdBindPipeline(VkCommandBuffer commandBuffer, VkPipe
set_pipeline_state(pipe_state);
skip |= validate_dual_src_blend_feature(dev_data, pipe_state);
addCommandBufferBinding(&pipe_state->cb_bindings, {HandleToUint64(pipeline), kVulkanObjectTypePipeline}, cb_state);
- if (VK_PIPELINE_BIND_POINT_GRAPHICS == pipelineBindPoint) {
- // Add binding for child renderpass
- auto rp_state = GetRenderPassState(dev_data, pipe_state->rp_state->renderPass);
- if (rp_state) {
- addCommandBufferBinding(&rp_state->cb_bindings, {HandleToUint64(rp_state->renderPass), kVulkanObjectTypeRenderPass},
- cb_state);
- }
- }
}
lock.unlock();
if (!skip) dev_data->dispatch_table.CmdBindPipeline(commandBuffer, pipelineBindPoint, pipeline);