summaryrefslogtreecommitdiffstats
path: root/src/glsl
diff options
context:
space:
mode:
authorChad Versace <[email protected]>2011-01-16 22:38:45 -0800
committerChad Versace <[email protected]>2011-01-17 09:41:25 -0800
commit889e1a5b6c6602198d649ea5881e0010dec575e9 (patch)
treed16e3b686f2d589ff82990603eaa1451d7ea505b /src/glsl
parent33279cd2d33a3f3804e4584f9134f4cc67567a54 (diff)
glsl: Add semantic checks for precision qualifiers
* Check that precision qualifiers only appear in language versions 1.00, 1.30, and later. * Check that precision qualifiers do not apply to bools and structs. Fixes the following Piglit tests: * spec/glsl-1.30/precision-qualifiers/precision-bool-01.frag * spec/glsl-1.30/precision-qualifiers/precision-struct-01.frag * spec/glsl-1.30/precision-qualifiers/precision-struct-02.frag
Diffstat (limited to 'src/glsl')
-rw-r--r--src/glsl/ast_to_hir.cpp33
1 files changed, 33 insertions, 0 deletions
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 51bc8ecd057..764c549005f 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@ -2351,6 +2351,39 @@ ast_declarator_list::hir(exec_list *instructions,
}
+ /* Precision qualifiers exists only in GLSL versions 1.00 and >= 1.30.
+ */
+ if (this->type->specifier->precision != ast_precision_none
+ && state->language_version != 100
+ && state->language_version < 130) {
+
+ _mesa_glsl_error(&loc, state,
+ "precision qualifiers are supported only in GLSL ES "
+ "1.00, and GLSL 1.30 and later");
+ }
+
+
+ /* Precision qualifiers do not apply to bools and structs.
+ *
+ * From section 4.5.2 of the GLSL 1.30 spec:
+ * "Any floating point or any integer declaration can have the type
+ * preceded by one of these precision qualifiers [...] Literal
+ * constants do not have precision qualifiers. Neither do Boolean
+ * variables.
+ */
+ if (this->type->specifier->precision != ast_precision_none
+ && this->type->specifier->type_specifier == ast_bool) {
+
+ _mesa_glsl_error(&loc, state,
+ "preicion qualifiers do not apply to type bool");
+ }
+ if (this->type->specifier->precision != ast_precision_none
+ && this->type->specifier->structure != NULL) {
+
+ _mesa_glsl_error(&loc, state,
+ "precision qualifiers do not apply to structures");
+ }
+
/* Process the initializer and add its instructions to a temporary
* list. This list will be added to the instruction stream (below) after
* the declaration is added. This is done because in some cases (such as