fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-07 00:38:48 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

broadcom/compiler: move nir_lower_undef_to_zero out of optimization loop

Browse source 
The combination of nir_opt_if and nir_lower_undef_to_zero running inside
the optimization loop could make it to not converge.

This was exercised by ollama running gemma3 compute shaders.

Removing the pass from the optimization loop results in No changes in
shader-db.

Assisted-by: Claude Opus 4.6
Fixes: cbe24a0e9c ("broadcom/compiler: use nir_lower_undef_to_zero")
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/41256>
This commit is contained in:
Jose Maria Casanova Crespo 2026-04-15 01:07:54 +02:00
parent b4591f4b30
commit 2cd51a6efc
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/broadcom/compiler/nir_to_vir.c
View file

@ -2261,7 +2261,6 @@ v3d_optimize_nir(struct v3d_compile *c, struct nir_shader *s)
}
NIR_PASS(progress, s, nir_opt_undef);
NIR_PASS(progress, s, nir_lower_undef_to_zero);
if (c && !c->disable_loop_unrolling &&
s->options->max_unroll_iterations > 0) {
@ -2272,6 +2271,8 @@ v3d_optimize_nir(struct v3d_compile *c, struct nir_shader *s)
}
} while (progress);
NIR_PASS(progress, s, nir_lower_undef_to_zero);
/* needs to be outside of optimization loop, otherwise it fights with
* opt_algebraic optimizing the conversion lowering
*/