Mesa (master): glsl: binding point is a texture unit, which is a combined space

Ilia Mirkin imirkin at kemper.freedesktop.org
Tue Jun 23 16:11:01 UTC 2015


Module: Mesa
Branch: master
Commit: fccf012adc0d3aad877de095244324aa1d2d046a
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fccf012adc0d3aad877de095244324aa1d2d046a

Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Tue Jun 23 00:16:59 2015 -0400

glsl: binding point is a texture unit, which is a combined space

This fixes compilation failures in Dota 2 Reborn where a texture unit
binding point was used that was numerically higher than the max
per stage.

Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>
Reviewed-by: Timothy Arceri <t_arceri at yahoo.com.au>
Tested-by: Nick Sarnie <commendsarnex at gmail.com>
Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>

---

 src/glsl/ast_to_hir.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 259e01e..8cb46be 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@ -2086,7 +2086,7 @@ validate_binding_qualifier(struct _mesa_glsl_parse_state *state,
        *  with an array of size N, all elements of the array from binding
        *  through binding + N - 1 must be within this range."
        */
-      unsigned limit = ctx->Const.Program[state->stage].MaxTextureImageUnits;
+      unsigned limit = ctx->Const.MaxCombinedTextureImageUnits;
 
       if (max_index >= limit) {
          _mesa_glsl_error(loc, state, "layout(binding = %d) for %d samplers "




More information about the mesa-commit mailing list