linker: Update some comments, remove a couple FINISHMEs
authorIan Romanick <ian.d.romanick@intel.com>
Wed, 23 Jun 2010 19:18:21 +0000 (12:18 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Wed, 23 Jun 2010 19:18:21 +0000 (12:18 -0700)
linker.cpp

index ebb1a81e7c9e1f2f7ef7f555a0620cefe5ecedd2..abf5371ce901f658b5e3cfe7e5c1fea3f8cb0bc6 100644 (file)
@@ -814,7 +814,6 @@ link_shaders(struct glsl_program *prog)
       goto done;
 
 
-   /* FINISHME: Perform inter-stage linking. */
    prog->_LinkedShaders = (struct glsl_shader **)
       calloc(2, sizeof(struct glsl_shader *));
    prog->_NumLinkedShaders = 0;
@@ -829,6 +828,10 @@ link_shaders(struct glsl_program *prog)
       prog->_NumLinkedShaders++;
    }
 
+   /* Here begins the inter-stage linking phase.  Some initial validation is
+    * performed, then locations are assigned for uniforms, attributes, and
+    * varyings.
+    */
    if (cross_validate_uniforms(prog)) {
       /* Validate the inputs of each stage with the output of the preceeding
        * stage.
@@ -862,8 +865,6 @@ link_shaders(struct glsl_program *prog)
 
    /* FINISHME: Assign fragment shader output locations. */
 
-   /* FINISHME: Generate code here. */
-
 done:
    free(vert_shader_list);
 }