Mesa (master): radeonsi/nir: Remove hack for builtins
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri May 31 09:16:05 UTC 2019
Module: Mesa
Branch: master
Commit: 78f33620e85fa3479bdded79981ae541871bfd2e
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=78f33620e85fa3479bdded79981ae541871bfd2e
Author: Connor Abbott <cwabbott0 at gmail.com>
Date: Wed May 29 16:03:25 2019 +0200
radeonsi/nir: Remove hack for builtins
We now bounds check properly in the uniform loading fast path, so
there's no need to disable it by pretending there are other UBO bindings
in use. The way this looks at the variable name was causing problems
when two piglit shaders, one with a name that triggered the hack and one
that didn't, got hashed to the same thing after stripping out the names.
Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
---
src/gallium/drivers/radeonsi/si_shader_nir.c | 13 ++-----------
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader_nir.c b/src/gallium/drivers/radeonsi/si_shader_nir.c
index 3a5e17c6b29..19ed71ae05d 100644
--- a/src/gallium/drivers/radeonsi/si_shader_nir.c
+++ b/src/gallium/drivers/radeonsi/si_shader_nir.c
@@ -779,17 +779,8 @@ void si_nir_scan_shader(const struct nir_shader *nir,
u_bit_consecutive(variable->data.binding, aoa_size);
}
} else if (base_type != GLSL_TYPE_ATOMIC_UINT) {
- if (strncmp(variable->name, "state.", 6) == 0 ||
- strncmp(variable->name, "gl_", 3) == 0) {
- /* FIXME: figure out why piglit tests with builtin
- * uniforms are failing without this.
- */
- info->const_buffers_declared =
- u_bit_consecutive(0, SI_NUM_CONST_BUFFERS);
- } else {
- info->const_buffers_declared |= 1;
- info->const_file_max[0] = max_slot;
- }
+ info->const_buffers_declared |= 1;
+ info->const_file_max[0] = max_slot;
}
}
More information about the mesa-commit
mailing list