Module: Mesa Branch: main Commit: 7fce0a5598fbfba73e10460b37865fe54b7c5bfc URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=7fce0a5598fbfba73e10460b37865fe54b7c5bfc
Author: Ian Romanick <[email protected]> Date: Thu Nov 30 13:30:53 2023 -0800 nir: Handle divergence for decl_reg Once decl_reg is handled, src[0].ssa->divergent will be properly set, so load_reg and load_reg_indirect do not need special treatment. shader-db can run to completion on HSW, IVB, and SNB now. No other testing was done. v2: Refactor nir_intrinsic_load_reg and nir_intrinsic_load_reg_indirect handling. Suggested by Daniel Schürmann. Reviewed-by: Daniel Schürmann <[email protected]> Fixes: 4fd257d20fe ("nir: Properly handle divergence for load_reg") Fixes: 6dbb5f1e07b ("intel/fs: rerun divergence analysis prior to convert_from_ssa") Closes: #10233 Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26436> --- src/compiler/nir/nir_divergence_analysis.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/compiler/nir/nir_divergence_analysis.c b/src/compiler/nir/nir_divergence_analysis.c index 047da6a2037..3d21d89125f 100644 --- a/src/compiler/nir/nir_divergence_analysis.c +++ b/src/compiler/nir/nir_divergence_analysis.c @@ -219,14 +219,9 @@ visit_intrinsic(nir_shader *shader, nir_intrinsic_instr *instr) is_divergent = false; break; - case nir_intrinsic_load_reg: - case nir_intrinsic_load_reg_indirect: { - nir_intrinsic_instr *decl = nir_reg_get_decl(instr->src[0].ssa); - is_divergent = nir_intrinsic_divergent(decl); - if (instr->intrinsic == nir_intrinsic_load_reg_indirect) - is_divergent |= instr->src[1].ssa->divergent; + case nir_intrinsic_decl_reg: + is_divergent = nir_intrinsic_divergent(instr); break; - } /* Intrinsics with divergence depending on shader stage and hardware */ case nir_intrinsic_load_shader_record_ptr: @@ -465,7 +460,9 @@ visit_intrinsic(nir_shader *shader, nir_intrinsic_instr *instr) case nir_intrinsic_load_desc_set_dynamic_index_intel: case nir_intrinsic_load_global_constant_bounded: case nir_intrinsic_load_global_constant_offset: - case nir_intrinsic_resource_intel: { + case nir_intrinsic_resource_intel: + case nir_intrinsic_load_reg: + case nir_intrinsic_load_reg_indirect: { unsigned num_srcs = nir_intrinsic_infos[instr->intrinsic].num_srcs; for (unsigned i = 0; i < num_srcs; i++) { if (instr->src[i].ssa->divergent) {
