ir_reader: Don't initialize globals, builtins, or constructors.
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 22 Apr 2010 00:50:16 +0000 (17:50 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Thu, 29 Apr 2010 01:14:54 +0000 (18:14 -0700)
All of these are currently emitted as part of the IR, so by initializing
them, we actually end up with two copies.  For constructors, we may
eventually wish to avoid emitting them as part of the IR output.

ir_reader.cpp

index 8e470e95d26cfb7eac62c2f878a9cd942cdef010..eb7b7fcb6b3592bb79e073c4903d0b007e36dbdb 100644 (file)
@@ -63,9 +63,11 @@ _mesa_glsl_read_ir(_mesa_glsl_parse_state *state, exec_list *instructions,
    printf("\n-------------\n");
    
    _mesa_glsl_initialize_types(state);
-   _mesa_glsl_initialize_variables(instructions, state);
-   _mesa_glsl_initialize_constructors(instructions, state);
-   _mesa_glsl_initialize_functions(instructions, state);
+
+   /* FINISHME: Constructors probably shouldn't be emitted as part of the IR.
+    * FINISHME: Once they're not, remake them by calling:
+    * FINISHME: _mesa_glsl_initialize_constructors(instructions, state);
+    */
 
    read_instructions(state, instructions, expr, NULL);
 }