From: Brian Paul <brian.paul@tungstengraphics.com>
Date: Mon, 15 Dec 2008 23:49:32 +0000 (-0700)
Subject: mesa: added comments, remove unused code
X-Git-Tag: mesa-7.8~4139^2~390^2~41
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5b5b05c894bae7e82ef87eb06f63dc05ded9378a;p=platform%2Fupstream%2Fmesa.git

mesa: added comments, remove unused code
(cherry picked from commit a66ff046cc169c6479b00b1e7fc5d87b93ad60ed)
---

diff --git a/src/mesa/shader/slang/slang_codegen.c b/src/mesa/shader/slang/slang_codegen.c
index f3e56bb..5d647e7 100644
--- a/src/mesa/shader/slang/slang_codegen.c
+++ b/src/mesa/shader/slang/slang_codegen.c
@@ -2707,7 +2707,15 @@ make_constant_array(slang_assemble_ctx *A,
 
 
 /**
- * Generate IR node for allocating/declaring a variable.
+ * Generate IR node for allocating/declaring a variable (either a local or
+ * a global).
+ * Generally, this involves allocating slang_ir_storage for the variable,
+ * choosing a register file (temporary, constant, etc).  For ordinary
+ * variables we do not yet allocate storage though.  We do that when we
+ * find the first actual use of the variable to avoid allocating temp regs
+ * that will never get used.
+ * At this time, uniforms are always allocated space in this function.
+ *
  * \param initializer  Optional initializer expression for the variable.
  */
 static slang_ir_node *
@@ -4056,40 +4064,9 @@ _slang_codegen_global_variable(slang_assemble_ctx *A, slang_variable *var,
          }
          else {
             /* non-struct uniform */
-#if 01
             if (!_slang_gen_var_decl(A, var, var->initializer))
                return GL_FALSE;
             store = var->store;
-#else
-            GLint uniformLoc;
-            const GLfloat *initialValues = NULL;
-#if 0
-            /* this code needs some work yet */
-            if (make_constant_array(A, var, var->initializer)) {
-               /* OK */
-            }
-            else
-#endif
-            if (var->initializer) {
-               _slang_simplify(var->initializer, &A->space, A->atoms);
-               if (var->initializer->type == SLANG_OPER_LITERAL_FLOAT ||
-                   var->initializer->type == SLANG_OPER_LITERAL_INT) {
-                  /* simple float/vector initializer */
-                  initialValues = var->initializer->literal;
-               }
-               else {
-                  /* complex initializer */
-                  slang_info_log_error(A->log,
-                     "unsupported initializer for uniform '%s'", varName);
-                  return GL_FALSE;
-               }
-            }
-
-            uniformLoc = _mesa_add_uniform(prog->Parameters, varName,
-                                           totalSize, datatype, initialValues);
-            store = _slang_new_ir_storage_swz(PROGRAM_UNIFORM, uniformLoc,
-                                              totalSize, swizzle);
-#endif
          }
       }
       else {