From 5ceedefd6c32fa31e6a35831a8a7a315e009ccc3 Mon Sep 17 00:00:00 2001 From: Timothy Arceri Date: Wed, 9 Nov 2016 23:50:07 +1100 Subject: [PATCH] mesa/glsl: remove hack to reset sampler units to zero Now that we have the is_arb_asm flag we can just skip the initialisation. V2: remove hack from standalone compiler where it was never needed since it only compiles glsl shaders. Reviewed-by: Eric Anholt --- src/compiler/glsl/link_uniforms.cpp | 11 ----------- src/compiler/glsl/standalone.cpp | 4 ---- src/mesa/program/program.c | 21 ++++++++++++++++----- 3 files changed, 16 insertions(+), 20 deletions(-) diff --git a/src/compiler/glsl/link_uniforms.cpp b/src/compiler/glsl/link_uniforms.cpp index 90eac5c..caee147 100644 --- a/src/compiler/glsl/link_uniforms.cpp +++ b/src/compiler/glsl/link_uniforms.cpp @@ -1299,17 +1299,6 @@ link_assign_uniform_locations(struct gl_shader_program *prog, if (sh == NULL) continue; - /* Uniforms that lack an initializer in the shader code have an initial - * value of zero. This includes sampler uniforms. - * - * Page 24 (page 30 of the PDF) of the GLSL 1.20 spec says: - * - * "The link time initial value is either the value of the variable's - * initializer, if present, or 0 if no initializer is present. Sampler - * types cannot have initializers." - */ - memset(sh->SamplerUnits, 0, sizeof(sh->SamplerUnits)); - link_update_uniform_buffer_variables(sh, i); /* Reset various per-shader target counts. diff --git a/src/compiler/glsl/standalone.cpp b/src/compiler/glsl/standalone.cpp index e93e9af..9a8d75d 100644 --- a/src/compiler/glsl/standalone.cpp +++ b/src/compiler/glsl/standalone.cpp @@ -107,10 +107,6 @@ init_gl_program(struct gl_program *prog, GLenum target, bool is_arb_asm) prog->RefCount = 1; prog->Format = GL_PROGRAM_FORMAT_ASCII_ARB; prog->is_arb_asm = is_arb_asm; - - /* default mapping from samplers to texture units */ - for (int i = 0; i < MAX_SAMPLERS; i++) - prog->SamplerUnits[i] = i; } struct gl_program * diff --git a/src/mesa/program/program.c b/src/mesa/program/program.c index 3edeec8..0e499c6 100644 --- a/src/mesa/program/program.c +++ b/src/mesa/program/program.c @@ -181,8 +181,6 @@ struct gl_program * _mesa_init_gl_program(struct gl_program *prog, GLenum target, GLuint id, bool is_arb_asm) { - GLuint i; - if (!prog) return NULL; @@ -195,9 +193,22 @@ _mesa_init_gl_program(struct gl_program *prog, GLenum target, GLuint id, prog->info.stage = _mesa_program_enum_to_shader_stage(target); prog->is_arb_asm = is_arb_asm; - /* default mapping from samplers to texture units */ - for (i = 0; i < MAX_SAMPLERS; i++) - prog->SamplerUnits[i] = i; + /* Uniforms that lack an initializer in the shader code have an initial + * value of zero. This includes sampler uniforms. + * + * Page 24 (page 30 of the PDF) of the GLSL 1.20 spec says: + * + * "The link time initial value is either the value of the variable's + * initializer, if present, or 0 if no initializer is present. Sampler + * types cannot have initializers." + * + * So we only initialise ARB assembly style programs. + */ + if (is_arb_asm) { + /* default mapping from samplers to texture units */ + for (unsigned i = 0; i < MAX_SAMPLERS; i++) + prog->SamplerUnits[i] = i; + } return prog; } -- 2.7.4