From: Ian Romanick <[email protected]>

GLSL ES 3.00 adds gl_MinProgramTexelOffset with a negative value.
This constant must be -8 or less.

v2: Select "<=" comparison based on "gl_Min" name instead of "gl_Max".
The original commit message suggested this, and Eric seconded it.

Signed-off-by: Ian Romanick <[email protected]>
---
 tests/shaders/built-in-constants.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/tests/shaders/built-in-constants.c 
b/tests/shaders/built-in-constants.c
index 8a534f7..af87107 100644
--- a/tests/shaders/built-in-constants.c
+++ b/tests/shaders/built-in-constants.c
@@ -41,7 +41,7 @@ char *required_glsl_version_string = NULL;
 
 
 static const char *const uniform_template =
-       "uniform float f[%s >= %d ? 1 : -1];\n"
+       "uniform float f[%s %s %d ? 1 : -1];\n"
        ;
 
 static const char *const vertex_shader_body =
@@ -133,6 +133,7 @@ parse_file(const char *filename)
         * major.minor
         * gl_MaxFoo 8
         * gl_MaxBar 16
+        * gl_MinAsdf -2
         */
 
        /* Process the version requirement.
@@ -156,7 +157,8 @@ parse_file(const char *filename)
 
                line = (char *) eat_whitespace(line);
 
-               if (string_match("gl_Max", line) != 0) {
+               if (string_match("gl_Max", line) != 0
+                   && string_match("gl_Min", line) != 0) {
                        char bad_name[80];
 
                        strcpy_to_space(bad_name, line);
@@ -273,13 +275,15 @@ piglit_init(int argc, char **argv)
 
        for (i = 0; i < num_tests; i++) {
                bool subtest_pass = true;
+               const char *comparitor =
+                       string_match("gl_Min", tests[i].name) ? "<=" : ">=";
 
                /* Generate the uniform declaration for the test.  This will
                 * be shared by all shader stages.
                 */
                snprintf(uniform, sizeof(uniform),
                         uniform_template,
-                        tests[i].name, tests[i].minimum);
+                        tests[i].name, comparitor, tests[i].minimum);
 
                /* Try to compile the vertex shader.
                 */
-- 
1.8.1.4

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to