anv: remove redundant code calculating dynamic states mask
authorMarcin Ślusarz <marcin.slusarz@intel.com>
Tue, 24 May 2022 10:49:14 +0000 (12:49 +0200)
committerMarge Bot <emma+marge@anholt.net>
Mon, 30 May 2022 11:46:13 +0000 (11:46 +0000)
pipeline->dynamic_states is already set by anv_graphics_pipeline_init
since 231651fd89fb007610568b3ca76837253e7683ff.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16773>

src/intel/vulkan/anv_pipeline.c
src/intel/vulkan/genX_pipeline.c

index 246a56c..6587fa0 100644 (file)
@@ -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++) {
index 192fec6..e5071db 100644 (file)
@@ -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.