From 34b5a717c0d264f1f18a951ab3fbbf0d9c7ea66b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marcin=20=C5=9Alusarz?= Date: Tue, 24 May 2022 12:49:14 +0200 Subject: [PATCH] anv: remove redundant code calculating dynamic states mask pipeline->dynamic_states is already set by anv_graphics_pipeline_init since 231651fd89fb007610568b3ca76837253e7683ff. Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/vulkan/anv_pipeline.c | 10 +--------- src/intel/vulkan/genX_pipeline.c | 11 +---------- 2 files changed, 2 insertions(+), 19 deletions(-) diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c index 246a56ce962..6587fa0651c 100644 --- a/src/intel/vulkan/anv_pipeline.c +++ b/src/intel/vulkan/anv_pipeline.c @@ -1421,15 +1421,7 @@ anv_pipeline_compile_graphics(struct anv_graphics_pipeline *pipeline, const struct brw_compiler *compiler = pipeline->base.device->physical->compiler; struct anv_pipeline_stage stages[ANV_GRAPHICS_SHADER_STAGE_COUNT] = {}; - /* Information on which states are considered dynamic. */ - const VkPipelineDynamicStateCreateInfo *dyn_info = - info->pDynamicState; - uint32_t dynamic_states = 0; - if (dyn_info) { - for (unsigned i = 0; i < dyn_info->dynamicStateCount; i++) - dynamic_states |= - anv_cmd_dirty_bit_for_vk_dynamic_state(dyn_info->pDynamicStates[i]); - } + uint32_t dynamic_states = pipeline->dynamic_states; VkResult result; for (uint32_t i = 0; i < info->stageCount; i++) { diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c index 192fec6b6e3..e5071dbaa60 100644 --- a/src/intel/vulkan/genX_pipeline.c +++ b/src/intel/vulkan/genX_pipeline.c @@ -2713,16 +2713,7 @@ genX(graphics_pipeline_create)( return result; } - /* Information on which states are considered dynamic. */ - const VkPipelineDynamicStateCreateInfo *dyn_info = - pCreateInfo->pDynamicState; - uint32_t dynamic_states = 0; - if (dyn_info) { - for (unsigned i = 0; i < dyn_info->dynamicStateCount; i++) - dynamic_states |= - anv_cmd_dirty_bit_for_vk_dynamic_state(dyn_info->pDynamicStates[i]); - } - + uint32_t dynamic_states = pipeline->dynamic_states; /* If rasterization is not enabled, various CreateInfo structs must be * ignored.