From a63a31a47b5e196bf32217f0a2c52f33530a68ed Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Wed, 28 Sep 2022 18:19:19 +0200 Subject: [PATCH] radv: remove NULL checks when binding a graphics pipeline The driver no longer bind NULL graphics pipelines, so these checks are useless. Signed-off-by: Samuel Pitoiset Reviewed-By: Mike Blumenkrantz Part-of: --- src/amd/vulkan/radv_cmd_buffer.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 7ced8d7eae0..60af72a3710 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -5146,15 +5146,14 @@ radv_CmdBindPipeline(VkCommandBuffer commandBuffer, VkPipelineBindPoint pipeline break; } case VK_PIPELINE_BIND_POINT_GRAPHICS: { - struct radv_graphics_pipeline *graphics_pipeline = - pipeline ? radv_pipeline_to_graphics(pipeline) : NULL; + struct radv_graphics_pipeline *graphics_pipeline = radv_pipeline_to_graphics(pipeline); if (cmd_buffer->state.graphics_pipeline == graphics_pipeline) return; radv_mark_descriptor_sets_dirty(cmd_buffer, pipelineBindPoint); bool vtx_emit_count_changed = - !pipeline || !cmd_buffer->state.graphics_pipeline || + !cmd_buffer->state.graphics_pipeline || cmd_buffer->state.graphics_pipeline->vtx_emit_num != graphics_pipeline->vtx_emit_num || cmd_buffer->state.graphics_pipeline->vtx_base_sgpr != graphics_pipeline->vtx_base_sgpr; cmd_buffer->state.graphics_pipeline = graphics_pipeline;