Bump GL_MAX_TEXTURE_COORDS from 2 to 4
authorIan Romanick <ian.d.romanick@intel.com>
Tue, 8 Jun 2010 02:10:33 +0000 (19:10 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Tue, 8 Jun 2010 02:10:33 +0000 (19:10 -0700)
Every platform that supports GLSL sets GL_MAX_TEXTURE_COORDS to at
least 4, so hard-code 4 for now.

This causes the following tests to pass:

    glslparsertest/glsl2/norsetto-bumptbn_sh_fp.vert
    glslparsertest/glsl2/xreal-lighting-d-omni.vert
    glslparsertest/glsl2/xreal-lighting-db-omni.vert
    glslparsertest/glsl2/xreal-lighting-dbs-omni.vert

ir_variable.cpp

index af8ad2c..0c0d127 100644 (file)
@@ -97,11 +97,12 @@ generate_110_uniforms(exec_list *instructions,
    }
 
    /* FINISHME: The size of this array is implementation dependent based on the
-    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  GL_MAX_TEXTURE_COORDS must be
-    * FINISHME: at least 2, so hard-code 2 for now.
+    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  Every platform that supports
+    * FINISHME: GLSL sets GL_MAX_TEXTURE_COORDS to at least 4, so hard-code 4
+    * FINISHME: for now.
     */
    const glsl_type *const mat4_array_type =
-      glsl_type::get_array_instance(glsl_type::mat4_type, 2);
+      glsl_type::get_array_instance(glsl_type::mat4_type, 4);
 
    add_variable("gl_TextureMatrix", ir_var_uniform, mat4_array_type,
                instructions, symtab);
@@ -149,11 +150,12 @@ generate_110_vs_variables(exec_list *instructions,
    generate_110_uniforms(instructions, symtab);
 
    /* FINISHME: The size of this array is implementation dependent based on the
-    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  GL_MAX_TEXTURE_COORDS must be
-    * FINISHME: at least 2, so hard-code 2 for now.
+    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  Every platform that supports
+    * FINISHME: GLSL sets GL_MAX_TEXTURE_COORDS to at least 4, so hard-code 4
+    * FINISHME: for now.
     */
    const glsl_type *const vec4_array_type =
-      glsl_type::get_array_instance(glsl_type::vec4_type, 2);
+      glsl_type::get_array_instance(glsl_type::vec4_type, 4);
 
    add_variable("gl_TexCoord", ir_var_out, vec4_array_type, instructions,
                symtab);
@@ -229,11 +231,12 @@ generate_110_fs_variables(exec_list *instructions,
    generate_110_uniforms(instructions, symtab);
 
    /* FINISHME: The size of this array is implementation dependent based on the
-    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  GL_MAX_TEXTURE_COORDS must be
-    * FINISHME: at least 2, so hard-code 2 for now.
+    * FINISHME: value of GL_MAX_TEXTURE_COORDS.  Every platform that supports
+    * FINISHME: GLSL sets GL_MAX_TEXTURE_COORDS to at least 4, so hard-code 4
+    * FINISHME: for now.
     */
    const glsl_type *const vec4_array_type =
-      glsl_type::get_array_instance(glsl_type::vec4_type, 2);
+      glsl_type::get_array_instance(glsl_type::vec4_type, 4);
 
    add_variable("gl_TexCoord", ir_var_in, vec4_array_type, instructions,
                symtab);