There was a typo on commit c0cd5b, doing it when explicit_binding was false. This prevented to use any binding point different to 0.
Cc: 10.4, 10.5 <mesa-sta...@lists.freedesktop.org> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90175 --- Piglit test to catch this error proposed here: http://lists.freedesktop.org/archives/piglit/2015-April/015877.html src/glsl/link_atomics.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/link_atomics.cpp b/src/glsl/link_atomics.cpp index 603873a..9528e42 100644 --- a/src/glsl/link_atomics.cpp +++ b/src/glsl/link_atomics.cpp @@ -201,7 +201,7 @@ link_assign_atomic_counter_resources(struct gl_context *ctx, gl_uniform_storage *const storage = &prog->UniformStorage[id]; mab.Uniforms[j] = id; - if (!var->data.explicit_binding) + if (var->data.explicit_binding) var->data.binding = i; storage->atomic_buffer_index = i; -- 2.1.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
