nir/lower_tex: Make the adding a 0 LOD to nir_op_tex in the VS optional.

This controls the whole lowering of "make tex ops with implicit
derivatives on non-implicit-derivative stages be tex ops with an explicit
lod of 0 instead", but it's really hard to describe that in a git commit
summary.

All existing callers get it added except:
- nir_to_tgsi which didn't want it.
- nouveau, which didn't want it (fixes regressions in shadowcube and
  shadow2darray with NIR, since the shading languages don't expose txl of
  those sampler types and thus it's not supported in HW)
- optional lowering passes in mesa/st (lower_rect, YUV lowering, etc)

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Jason Ekstrand <jason.ekstrand@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16156>
This commit is contained in:
Emma Anholt 2022-04-25 16:55:45 -07:00 committed by Marge Bot
parent fd90a57e28
commit 536c8ee96d
21 changed files with 35 additions and 6 deletions

View File

@ -795,6 +795,7 @@ radv_shader_compile_to_nir(struct radv_device *device, const struct radv_pipelin
.lower_txs_cube_array = true,
.lower_to_fragment_fetch_amd = true,
.lower_lod_zero_width = true,
.lower_invalid_implicit_lod = true,
};
nir_lower_tex(nir, &tex_options);

View File

@ -1517,6 +1517,7 @@ agx_compile_shader_nir(nir_shader *nir,
nir_lower_tex_options lower_tex_options = {
.lower_txs_lod = true,
.lower_txp = ~0,
.lower_invalid_implicit_lod = true,
};
nir_tex_src_type_constraints tex_constraints = {

View File

@ -613,6 +613,7 @@ v3d_lower_nir(struct v3d_compile *c)
.lower_txp = ~0,
/* Apply swizzles to all samplers. */
.swizzle_result = ~0,
.lower_invalid_implicit_lod = true,
};
/* Lower the format swizzle and (for 32-bit returns)

View File

@ -5022,6 +5022,12 @@ typedef struct nir_lower_tex_options {
*/
bool lower_lod_zero_width;
/* Turns nir_op_tex and other ops with an implicit derivative, in stages
* without implicit derivatives (like the vertex shader) to have an explicit
* LOD with a value of 0.
*/
bool lower_invalid_implicit_lod;
/**
* Payload data to be sent to callback / filter functions.
*/

View File

@ -1480,7 +1480,8 @@ nir_lower_tex_block(nir_block *block, nir_builder *b,
* use an explicit LOD of 0.
* But don't touch RECT samplers because they don't have mips.
*/
if (nir_tex_instr_has_implicit_derivative(tex) &&
if (options->lower_invalid_implicit_lod &&
nir_tex_instr_has_implicit_derivative(tex) &&
tex->sampler_dim != GLSL_SAMPLER_DIM_RECT &&
!nir_shader_supports_implicit_lod(b->shader)) {
lower_zero_lod(b, tex);

View File

@ -270,6 +270,7 @@ ir3_finalize_nir(struct ir3_compiler *compiler, nir_shader *s)
struct nir_lower_tex_options tex_options = {
.lower_rect = 0,
.lower_tg4_offsets = true,
.lower_invalid_implicit_lod = true,
};
if (compiler->gen >= 4) {

View File

@ -2497,6 +2497,7 @@ void lp_build_opt_nir(struct nir_shader *nir)
static const struct nir_lower_tex_options lower_tex_options = {
.lower_tg4_offsets = true,
.lower_txp = ~0u,
.lower_invalid_implicit_lod = true,
};
NIR_PASS_V(nir, nir_lower_tex, &lower_tex_options);
NIR_PASS_V(nir, nir_lower_frexp);
@ -2509,7 +2510,7 @@ void lp_build_opt_nir(struct nir_shader *nir)
NIR_PASS(progress, nir, nir_opt_algebraic);
NIR_PASS(progress, nir, nir_lower_pack);
nir_lower_tex_options options = { 0, };
nir_lower_tex_options options = { .lower_invalid_implicit_lod = true, };
NIR_PASS_V(nir, nir_lower_tex, &options);
const nir_lower_subgroups_options subgroups_options = {

View File

@ -216,7 +216,9 @@ static void
crocus_lower_swizzles(struct nir_shader *nir,
const struct brw_sampler_prog_key_data *key_tex)
{
struct nir_lower_tex_options tex_options = { 0 };
struct nir_lower_tex_options tex_options = {
.lower_invalid_implicit_lod = true,
};
uint32_t mask = nir->info.textures_used[0];
while (mask) {

View File

@ -1082,6 +1082,7 @@ select_shader_variant(struct d3d12_selection_context *sel_ctx, d3d12_shader_sele
tex_options.saturate_s = key.tex_saturate_s;
tex_options.saturate_r = key.tex_saturate_r;
tex_options.saturate_t = key.tex_saturate_t;
tex_options.lower_invalid_implicit_lod = true;
NIR_PASS_V(new_nir_variant, nir_lower_tex, &tex_options);
}

View File

@ -1117,7 +1117,7 @@ etna_compile_shader(struct etna_shader_variant *v)
NIR_PASS_V(s, nir_lower_regs_to_ssa);
NIR_PASS_V(s, nir_lower_vars_to_ssa);
NIR_PASS_V(s, nir_lower_indirect_derefs, nir_var_all, UINT32_MAX);
NIR_PASS_V(s, nir_lower_tex, &(struct nir_lower_tex_options) { .lower_txp = ~0u });
NIR_PASS_V(s, nir_lower_tex, &(struct nir_lower_tex_options) { .lower_txp = ~0u, .lower_invalid_implicit_lod = true, });
if (v->key.has_sample_tex_compare)
NIR_PASS_V(s, nir_lower_tex_shadow, v->key.num_texture_states,

View File

@ -110,6 +110,7 @@ ir2_optimize_nir(nir_shader *s, bool lower)
struct nir_lower_tex_options tex_options = {
.lower_txp = ~0u,
.lower_rect = 0,
.lower_invalid_implicit_lod = true,
};
if (FD_DBG(DISASM)) {

View File

@ -287,6 +287,7 @@ lima_fs_compile_shader(struct lima_context *ctx,
struct nir_lower_tex_options tex_options = {
.swizzle_result = ~0u,
.lower_invalid_implicit_lod = true,
};
for (int i = 0; i < ARRAY_SIZE(key->tex); i++) {

View File

@ -214,6 +214,7 @@ main(int argc, char **argv)
struct nir_lower_tex_options tex_options = {
.lower_txp = ~0u,
.lower_invalid_implicit_lod = true,
};
nir_shader *nir = load_glsl(1, filename, stage);

View File

@ -836,6 +836,7 @@ int r600_shader_from_nir(struct r600_context *rctx,
struct nir_lower_tex_options lower_tex_options = {0};
lower_tex_options.lower_txp = ~0u;
lower_tex_options.lower_txf_offset = true;
lower_tex_options.lower_invalid_implicit_lod = true;
NIR_PASS_V(sel->nir, nir_lower_tex, &lower_tex_options);
NIR_PASS_V(sel->nir, r600::r600_nir_lower_txl_txf_array_or_cube);

View File

@ -234,6 +234,7 @@ static void si_lower_nir(struct si_screen *sscreen, struct nir_shader *nir)
static const struct nir_lower_tex_options lower_tex_options = {
.lower_txp = ~0u,
.lower_txs_cube_array = true,
.lower_invalid_implicit_lod = true,
};
NIR_PASS_V(nir, nir_lower_tex, &lower_tex_options);

View File

@ -2262,6 +2262,7 @@ vc4_shader_ntq(struct vc4_context *vc4, enum qstage stage,
/* Apply swizzles to all samplers. */
.swizzle_result = ~0,
.lower_invalid_implicit_lod = true,
};
/* Lower the format swizzle and ARB_texture_swizzle-style swizzle.

View File

@ -2184,7 +2184,9 @@ zink_shader_finalize(struct pipe_screen *pscreen, void *nirptr)
struct zink_screen *screen = zink_screen(pscreen);
nir_shader *nir = nirptr;
nir_lower_tex_options tex_opts = {0};
nir_lower_tex_options tex_opts = {
.lower_invalid_implicit_lod = true,
};
/*
Sampled Image must be an object whose type is OpTypeSampledImage.
The Dim operand of the underlying OpTypeImage must be 1D, 2D, 3D,

View File

@ -845,6 +845,7 @@ brw_preprocess_nir(const struct brw_compiler *compiler, nir_shader *nir,
.lower_txs_lod = true, /* Wa_14012320009 */
.lower_offset_filter =
devinfo->verx10 >= 125 ? lower_xehp_tg4_offset_filter : NULL,
.lower_invalid_implicit_lod = true,
};
OPT(nir_lower_tex, &tex_options);
@ -1308,6 +1309,7 @@ brw_nir_apply_sampler_key(nir_shader *nir,
nir_lower_tex_options tex_options = {
.lower_txd_clamp_bindless_sampler = true,
.lower_txd_clamp_if_sampler_index_not_lt_16 = true,
.lower_invalid_implicit_lod = true,
};
/* Iron Lake and prior require lowering of all rectangle textures */

View File

@ -718,7 +718,9 @@ spirv_to_dxil(const uint32_t *words, size_t word_count,
}
NIR_PASS_V(nir, nir_lower_readonly_images_to_tex, true);
nir_lower_tex_options lower_tex_options = {0};
nir_lower_tex_options lower_tex_options = {
.lower_invalid_implicit_lod = true,
};
NIR_PASS_V(nir, nir_lower_tex, &lower_tex_options);
NIR_PASS_V(nir, dxil_spirv_nir_fix_sample_mask_type);

View File

@ -4066,6 +4066,7 @@ bi_optimize_nir(nir_shader *nir, unsigned gpu_id, bool is_blend)
.lower_txp = ~0,
.lower_tg4_broadcom_swizzle = true,
.lower_txd = true,
.lower_invalid_implicit_lod = true,
};
NIR_PASS(progress, nir, pan_nir_lower_64bit_intrin);

View File

@ -346,6 +346,7 @@ optimise_nir(nir_shader *nir, unsigned quirks, bool is_blend, bool is_blit)
.lower_tg4_broadcom_swizzle = true,
/* TODO: we have native gradient.. */
.lower_txd = true,
.lower_invalid_implicit_lod = true,
};
NIR_PASS(progress, nir, nir_lower_tex, &lower_tex_options);