From e2fabb4e4a1616e7b0c2c6b9437503737a7d47c8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timur=20Krist=C3=B3f?= Date: Mon, 8 Dec 2025 12:48:21 -0600 Subject: [PATCH] radv: Don't call nir_opt_combine_stores anymore MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Also no need for nir_lower_tess_level_array_vars_to_vec. These should be now handled by nir_opt_vectorize_io. Fossil DB stats on Strix Halo (GFX11.5): Totals from 373 (0.47% of 79825) affected shaders: Instrs: 381930 -> 380786 (-0.30%); split: -0.30%, +0.00% CodeSize: 1888160 -> 1883644 (-0.24%); split: -0.24%, +0.01% Latency: 1008755 -> 1008053 (-0.07%); split: -0.08%, +0.01% InvThroughput: 156523 -> 155275 (-0.80%); split: -0.81%, +0.01% Copies: 22357 -> 20812 (-6.91%); split: -6.93%, +0.02% VALU: 240904 -> 239359 (-0.64%); split: -0.64%, +0.00% Signed-off-by: Timur Kristóf Reviewed-by: Samuel Pitoiset Reviewed-by: Georg Lehmann Part-of: --- src/amd/vulkan/radv_pipeline_graphics.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/amd/vulkan/radv_pipeline_graphics.c b/src/amd/vulkan/radv_pipeline_graphics.c index 07644fa2117..618d8fe15f9 100644 --- a/src/amd/vulkan/radv_pipeline_graphics.c +++ b/src/amd/vulkan/radv_pipeline_graphics.c @@ -1374,11 +1374,6 @@ radv_link_shaders(const struct radv_device *device, struct radv_shader_stage *pr (producer->info.stage == MESA_SHADER_VERTEX && has_geom_or_tess) || (producer->info.stage == MESA_SHADER_TESS_EVAL && merged_gs)) { NIR_PASS(_, producer, nir_opt_vectorize_io_vars, nir_var_shader_out); - - if (producer->info.stage == MESA_SHADER_TESS_CTRL) - NIR_PASS(_, producer, nir_lower_tess_level_array_vars_to_vec); - - NIR_PASS(_, producer, nir_opt_combine_stores, nir_var_shader_out); } if (consumer->info.stage == MESA_SHADER_GEOMETRY || consumer->info.stage == MESA_SHADER_TESS_CTRL ||