diff options
| author | Samuel Pitoiset <[email protected]> | 2017-10-04 22:27:38 +0200 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2017-10-04 23:18:48 +0200 |
| commit | b53c207659e65a29081c0f26b189f37fdd895406 (patch) | |
| tree | 72e494952a9b819e93d1e2327d37594a45c0de92 | |
| parent | 2572c2771d0cab0b9bc489d354ede44dfc88547b (diff) | |
radv: check that pipeline is different before binding it
We only need to dirty the descriptors when the pipeline is
a new one, because user SGPRs can be potentially different.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
| -rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 61ea11c12a6..4b41b358e99 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2454,14 +2454,20 @@ void radv_CmdBindPipeline( RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer); RADV_FROM_HANDLE(radv_pipeline, pipeline, _pipeline); - radv_mark_descriptor_sets_dirty(cmd_buffer); - switch (pipelineBindPoint) { case VK_PIPELINE_BIND_POINT_COMPUTE: + if (cmd_buffer->state.compute_pipeline == pipeline) + return; + radv_mark_descriptor_sets_dirty(cmd_buffer); + cmd_buffer->state.compute_pipeline = pipeline; cmd_buffer->push_constant_stages |= VK_SHADER_STAGE_COMPUTE_BIT; break; case VK_PIPELINE_BIND_POINT_GRAPHICS: + if (cmd_buffer->state.pipeline == pipeline) + return; + radv_mark_descriptor_sets_dirty(cmd_buffer); + cmd_buffer->state.pipeline = pipeline; if (!pipeline) break; |