glsl: Allow binding of image variables with 420pack.
authorMatt Turner <mattst88@gmail.com>
Mon, 7 Dec 2015 19:14:56 +0000 (11:14 -0800)
committerMatt Turner <mattst88@gmail.com>
Tue, 8 Dec 2015 23:36:57 +0000 (15:36 -0800)
This interaction was missed in the addition of ARB_image_load_store.

Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93266
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/glsl/ast_to_hir.cpp

index 8fcda63d95ec9f4f3b649d235a8c27f2608d1027..6d7e40fb2bb7f13a8be1a5f3cf0c8118312f43fe 100644 (file)
@@ -2649,7 +2649,9 @@ apply_explicit_binding(struct _mesa_glsl_parse_state *state,
 
          return;
       }
-   } else if (state->is_version(420, 310) && base_type->is_image()) {
+   } else if ((state->is_version(420, 310) ||
+               state->ARB_shading_language_420pack_enable) &&
+              base_type->is_image()) {
       assert(ctx->Const.MaxImageUnits <= MAX_IMAGE_UNITS);
       if (max_index >= ctx->Const.MaxImageUnits) {
          _mesa_glsl_error(loc, state, "Image binding %d exceeds the "