diff options
| author | Tobin Ehlis <tobine@google.com> | 2017-09-28 12:46:58 -0600 |
|---|---|---|
| committer | Tobin Ehlis <tobine@google.com> | 2017-09-28 13:26:43 -0700 |
| commit | 0bdbf3c2d36a556ab2aa9ccf27b62a6ef5d21016 (patch) | |
| tree | 8b468d186408aec6019c04a8fbe0eeca398d9670 /layers | |
| parent | d6a76d89728d9364decb0b45c30416dc0b3062c1 (diff) | |
| download | usermoji-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.cpp | 8 |
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); |
