GLSL 3.30 wholly includes the ARB_explicit_attrib_location
functionality.  Failing to set this flag meant we wouldn't handle the
layout qualifiers appropriately.

Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
Reported-by: Dan Ginsburg <d...@valvesoftware.com>
Cc: Ian Romanick <i...@freedesktop.org>
---
 src/glsl/glsl_parser_extras.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/glsl/glsl_parser_extras.cpp b/src/glsl/glsl_parser_extras.cpp
index cac5a18..aa3ad8d 100644
--- a/src/glsl/glsl_parser_extras.cpp
+++ b/src/glsl/glsl_parser_extras.cpp
@@ -304,7 +304,7 @@ _mesa_glsl_parse_state::process_version_directive(YYLTYPE 
*locp, int version,
       this->ARB_uniform_buffer_object_enable = true;
    }
 
-   if (this->language_version == 300 && this->es_shader) {
+   if (this->is_version(330, 300)) {
       this->ARB_explicit_attrib_location_enable = true;
    }
 }
-- 
1.8.3.4

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to