summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTimothy Arceri <tarceri@itsqueeze.com>2020-10-01 20:23:28 +1000
committerEric Engestrom <eric@engestrom.ch>2020-10-14 19:29:27 +0200
commit07ba04a35add2662910369b16eac1280f22e6969 (patch)
treeb0543de50fc5513376148f99ebe79f4eb42370cf /src
parent55eed08808c0ec5a7b984310f4d0bde40c2481b5 (diff)
glsl: don't duplicate state vars as uniforms in the NIR linker
The linker was adding all state vars as uniforms, doubling the storage size for shaders using only builtin uniforms, which increased CPU overhead for constant buffer uploads. When this code was originally ported from the GLSL IR linker we forgot to exclude builtins because the check was not done in the add_uniform_to_shader class but rather a check was done when passing variables to this class for processing. Fixes: 664e4a610dc8 ("glsl/nir: Fill in the Parameters in NIR linker") Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Tested-by: Marek Olšák <marek.olsak@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6958> (cherry picked from commit 038fcbcaed31b97f8f477f2496f8cf0a809b1892)
Diffstat (limited to 'src')
-rw-r--r--src/compiler/glsl/gl_nir_link_uniforms.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/compiler/glsl/gl_nir_link_uniforms.c b/src/compiler/glsl/gl_nir_link_uniforms.c
index c8d32bdf338..00005c20126 100644
--- a/src/compiler/glsl/gl_nir_link_uniforms.c
+++ b/src/compiler/glsl/gl_nir_link_uniforms.c
@@ -530,6 +530,12 @@ add_parameter(struct gl_uniform_storage *uniform,
const struct glsl_type *type,
struct nir_link_uniforms_state *state)
{
+ /* Builtin uniforms are backed by PROGRAM_STATE_VAR, so don't add them as
+ * uniforms.
+ */
+ if (uniform->builtin)
+ return;
+
if (!state->params || uniform->is_shader_storage ||
(glsl_contains_opaque(type) && !state->current_var->data.bindless))
return;