From: Neha Bhende Date: Tue, 17 Oct 2023 22:58:23 +0000 (-0700) Subject: ntt: lower indirect tesslevels in ntt X-Git-Tag: upstream/23.3.3~337 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e886ef760ca0764fd004a989d97f4cb0bbe6bedb;p=platform%2Fupstream%2Fmesa.git ntt: lower indirect tesslevels in ntt Tessellation shader which are using indirect addressing for tesslevels e.g gl_TessLevelOuter[gl_InvocationID] = tessLevelOuter; are crashing because gl_TessLevelOuter is now a compact array variable and nir expects a constant array index into the compact array variable. This patch handles such cases. This fixes MR 21940 Fixes: 84006587d7e5 ("glsl: Delete the lower_tess_level pass.") Tested with glretrace Reviewed-by: Charmaine Lee Reviewed-by: Emma Anholt Part-of: (cherry picked from commit 8cfb46e27dfb12acdb1a00f7687a43899d770cca) --- diff --git a/.pick_status.json b/.pick_status.json index e50dc6f..b26e0f5 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -1764,7 +1764,7 @@ "description": "ntt: lower indirect tesslevels in ntt", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "84006587d7e5b87526a9b83135393251afeef777", "notes": null diff --git a/src/gallium/auxiliary/nir/nir_to_tgsi.c b/src/gallium/auxiliary/nir/nir_to_tgsi.c index 521a643..4cf7fbc 100644 --- a/src/gallium/auxiliary/nir/nir_to_tgsi.c +++ b/src/gallium/auxiliary/nir/nir_to_tgsi.c @@ -3898,6 +3898,15 @@ const void *nir_to_tgsi_options(struct nir_shader *s, NIR_PASS_V(s, nir_remove_dead_variables, nir_var_shader_in, NULL); } + /* Lower tesslevel indirect derefs for tessellation shader. + * tesslevels are now a compact array variable and nir expects a constant + * array index into the compact array variable. + */ + if (s->info.stage == MESA_SHADER_TESS_CTRL || + s->info.stage == MESA_SHADER_TESS_EVAL) { + NIR_PASS_V(s, nir_lower_indirect_derefs, 0 , UINT32_MAX); + } + NIR_PASS_V(s, nir_lower_io, nir_var_shader_in | nir_var_shader_out, type_size, (nir_lower_io_options)0);