diff options
author | Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> | 2021-03-30 21:50:04 +0200 |
---|---|---|
committer | Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> | 2021-04-05 10:14:24 +0200 |
commit | e4ef5f043315e9f06bfb082b867dc08efe310d55 (patch) | |
tree | 265d7d1444018e849b15ad35f0984e51b1de2e91 | |
parent | 72c54713aaed464e871c7177298e76f0202be252 (diff) |
mesa/st: ignore texture_index if tex_instr has deref src
texture_index is meaningless when a tex_instr has deref src.
Use var->data.binding instead.
CC: mesa-stable
Reviewed-by: Jesse Natalie <jenatali@microsoft.com>
Acked-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9931>
-rw-r--r-- | src/mesa/state_tracker/st_nir_lower_tex_src_plane.c | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/src/mesa/state_tracker/st_nir_lower_tex_src_plane.c b/src/mesa/state_tracker/st_nir_lower_tex_src_plane.c index edd80fb5201..8daf19bdcf5 100644 --- a/src/mesa/state_tracker/st_nir_lower_tex_src_plane.c +++ b/src/mesa/state_tracker/st_nir_lower_tex_src_plane.c @@ -121,25 +121,26 @@ lower_tex_src_plane_block(nir_builder *b, lower_tex_src_state *state, nir_block if (plane[0].i32 > 0) { unsigned y_samp = tex->texture_index; + int tex_index = nir_tex_instr_src_index(tex, nir_tex_src_texture_deref); + if (tex_index >= 0) { + nir_deref_instr *deref = nir_src_as_deref(tex->src[tex_index].src); + y_samp = nir_deref_instr_get_variable(deref)->data.binding; + } - assume(tex->texture_index == tex->sampler_index); assume(((state->lower_3plane & (1 << y_samp)) && plane[0].i32 < 3) || (plane[0].i32 < 2)); - tex->texture_index = tex->sampler_index = - state->sampler_map[y_samp][plane[0].i32 - 1]; - - BITSET_SET(state->shader->info.textures_used, tex->texture_index); + unsigned u_v_samp = state->sampler_map[y_samp][plane[0].i32 - 1]; + BITSET_SET(state->shader->info.textures_used, u_v_samp); /* For drivers using PIPE_CAP_NIR_SAMPLERS_AS_DEREF, we need * to reference the correct sampler nir variable. */ - int tex_index = nir_tex_instr_src_index(tex, nir_tex_src_texture_deref); int samp_index = nir_tex_instr_src_index(tex, nir_tex_src_sampler_deref); if (tex_index >= 0 && samp_index >= 0) { b->cursor = nir_before_instr(&tex->instr); - nir_variable* samp = find_sampler(state, tex->sampler_index); + nir_variable* samp = find_sampler(state, u_v_samp); assert(samp); nir_deref_instr *tex_deref_instr = nir_build_deref_var(b, samp); @@ -151,6 +152,10 @@ lower_tex_src_plane_block(nir_builder *b, lower_tex_src_state *state, nir_block nir_instr_rewrite_src(&tex->instr, &tex->src[samp_index].src, nir_src_for_ssa(tex_deref)); + } else { + /* For others we need to update texture_index */ + assume(tex->texture_index == tex->sampler_index); + tex->texture_index = tex->sampler_index = u_v_samp; } } |