From 0422165d9a81e18f68007c61ae5df88630256eed Mon Sep 17 00:00:00 2001 From: Caio Oliveira Date: Thu, 23 Apr 2026 11:14:06 -0700 Subject: [PATCH] brw: Remove various unused fields These are a mix of fields whose last used was removed or fields that were never used, possibly because they remained in a patch while the rest of the code changed before landing. Reviewed-by: Kenneth Graunke Part-of: --- src/intel/compiler/brw/brw_compile_bs.cpp | 1 - src/intel/compiler/brw/brw_compiler.h | 21 ------------------- src/intel/compiler/brw/brw_thread_payload.cpp | 3 --- 3 files changed, 25 deletions(-) diff --git a/src/intel/compiler/brw/brw_compile_bs.cpp b/src/intel/compiler/brw/brw_compile_bs.cpp index 6718c8c395b..c28443c3115 100644 --- a/src/intel/compiler/brw/brw_compile_bs.cpp +++ b/src/intel/compiler/brw/brw_compile_bs.cpp @@ -148,7 +148,6 @@ brw_compile_bs(const struct brw_compiler *compiler, brw_prog_data_init(&prog_data->base, ¶ms->base); prog_data->max_stack_size = 0; - prog_data->num_resume_shaders = num_resume_shaders; brw_generator g(compiler, ¶ms->base, &prog_data->base, shader->info.stage); diff --git a/src/intel/compiler/brw/brw_compiler.h b/src/intel/compiler/brw/brw_compiler.h index 8deb32e7248..99a40e08c8a 100644 --- a/src/intel/compiler/brw/brw_compiler.h +++ b/src/intel/compiler/brw/brw_compiler.h @@ -495,9 +495,6 @@ struct brw_stage_prog_data { unsigned grf_used; uint32_t source_hash; - - /* Whether shader uses atomic operations. */ - bool uses_atomic_load_store; }; /** @@ -889,11 +886,6 @@ brw_cs_prog_data_prog_offset(const struct brw_cs_prog_data *prog_data, struct brw_bs_prog_data { struct brw_stage_prog_data base; - /** Whether inline push data is used to provide a 64bit pointer to push - * constants - */ - bool uses_inline_push_addr; - /** SIMD size of the root shader */ uint8_t simd_size; @@ -902,9 +894,6 @@ struct brw_bs_prog_data { /** Offset into the shader where the resume SBT is located */ uint32_t resume_sbt_offset; - - /** Number of resume shaders */ - uint32_t num_resume_shaders; }; #define BRW_VUE_HEADER_VARYING_MASK \ @@ -971,7 +960,6 @@ struct brw_vue_prog_data { bool include_vue_handles; unsigned urb_read_length; - unsigned total_grf; uint32_t clip_distance_mask; uint32_t cull_distance_mask; @@ -1025,12 +1013,6 @@ struct brw_tcs_prog_data /** Should the non-SINGLE_PATCH payload provide primitive ID? */ bool include_primitive_id; - /** Whether the tessellation domain is unknown at compile time - * - * Used with VK_EXT_shader_object - */ - bool dynamic_domain; - /** Number vertices in output patch */ int instances; @@ -1122,9 +1104,6 @@ struct brw_mue_map { /* Per vertex offset in bytes from the start of the MUE (32B aligned) */ uint32_t per_vertex_offset; - /* Size of the per vertex header (32B aligned) */ - uint32_t per_vertex_header_size; - /* Per vertex stride in bytes (32B aligned) */ uint32_t per_vertex_stride; diff --git a/src/intel/compiler/brw/brw_thread_payload.cpp b/src/intel/compiler/brw/brw_thread_payload.cpp index d399a30beb2..f7c908a11f0 100644 --- a/src/intel/compiler/brw/brw_thread_payload.cpp +++ b/src/intel/compiler/brw/brw_thread_payload.cpp @@ -420,8 +420,6 @@ brw_task_mesh_thread_payload::brw_task_mesh_thread_payload(brw_shader &v) brw_bs_thread_payload::brw_bs_thread_payload(const brw_shader &v) { - struct brw_bs_prog_data *prog_data = brw_bs_prog_data(v.prog_data); - unsigned r = 0; /* R0: Thread header. */ @@ -431,7 +429,6 @@ brw_bs_thread_payload::brw_bs_thread_payload(const brw_shader &v) r += reg_unit(v.devinfo); /* R2: Inline Parameter. Used for argument addresses. */ - prog_data->uses_inline_push_addr = v.key->uses_inline_push_addr; inline_parameter = brw_ud1_grf(r, 0); global_arg_ptr = brw_ud1_grf(r, 0); local_arg_ptr = brw_ud1_grf(r, 2);