glsl2: Emit array constructors inline.
[mesa.git] / src / glsl / ast_function.cpp
index f1ab6f0c5a1c3d8597baddc8d9c4b4622c311121..1982c83a430e31bced4e313af992ed476c231d67 100644 (file)
 #include "glsl_types.h"
 #include "ir.h"
 
+inline unsigned min(unsigned a, unsigned b)
+{
+   return (a < b) ? a : b;
+}
+
+static ir_rvalue *
+convert_component(ir_rvalue *src, const glsl_type *desired_type);
+
 static unsigned
 process_parameters(exec_list *instructions, exec_list *actual_parameters,
                   exec_list *parameters,
@@ -54,10 +62,9 @@ process_call(exec_list *instructions, ir_function *f,
             YYLTYPE *loc, exec_list *actual_parameters,
             struct _mesa_glsl_parse_state *state)
 {
-   void *ctx = talloc_parent(state);
+   void *ctx = state;
 
-   const ir_function_signature *sig =
-      f->matching_signature(actual_parameters);
+   ir_function_signature *sig = f->matching_signature(actual_parameters);
 
    /* The instructions param will be used when the FINISHMEs below are done */
    (void) instructions;
@@ -88,14 +95,40 @@ process_call(exec_list *instructions, ir_function *f,
            }
         }
 
+        if (formal->type->is_numeric() || formal->type->is_boolean()) {
+           ir_rvalue *converted = convert_component(actual, formal->type);
+           actual->replace_with(converted);
+        }
+
         actual_iter.next();
         formal_iter.next();
       }
 
-      /* FINISHME: The list of actual parameters needs to be modified to
-       * FINISHME: include any necessary conversions.
+      /* Always insert the call in the instruction stream, and return a deref
+       * of its return val if it returns a value, since we don't know if
+       * the rvalue is going to be assigned to anything or not.
        */
-      return new(ctx) ir_call(sig, actual_parameters);
+      ir_call *call = new(ctx) ir_call(sig, actual_parameters);
+      if (!sig->return_type->is_void()) {
+        ir_variable *var;
+        ir_dereference_variable *deref;
+
+        var = new(ctx) ir_variable(sig->return_type,
+                                   talloc_asprintf(ctx, "%s_retval",
+                                                   sig->function_name()),
+                                   ir_var_temporary);
+        instructions->push_tail(var);
+
+        deref = new(ctx) ir_dereference_variable(var);
+        ir_assignment *assign = new(ctx) ir_assignment(deref, call, NULL);
+        instructions->push_tail(assign);
+
+        deref = new(ctx) ir_dereference_variable(var);
+        return deref;
+      } else {
+        instructions->push_tail(call);
+        return NULL;
+      }
    } else {
       /* FINISHME: Log a better error message here.  G++ will show the types
        * FINISHME: of the actual parameters and the set of candidate
@@ -114,7 +147,7 @@ match_function_by_name(exec_list *instructions, const char *name,
                       YYLTYPE *loc, exec_list *actual_parameters,
                       struct _mesa_glsl_parse_state *state)
 {
-   void *ctx = talloc_parent(state);
+   void *ctx = state;
    ir_function *f = state->symbols->get_function(name);
 
    if (f == NULL) {
@@ -131,6 +164,9 @@ match_function_by_name(exec_list *instructions, const char *name,
 
 /**
  * Perform automatic type conversion of constructor parameters
+ *
+ * This implements the rules in the "Conversion and Scalar Constructors"
+ * section (GLSL 1.10 section 5.4.1), not the "Implicit Conversions" rules.
  */
 static ir_rvalue *
 convert_component(ir_rvalue *src, const glsl_type *desired_type)
@@ -187,11 +223,11 @@ convert_component(ir_rvalue *src, const glsl_type *desired_type)
 
    assert(result != NULL);
 
+   /* Try constant folding; it may fold in the conversion we just added. */
    ir_constant *const constant = result->constant_expression_value();
    return (constant != NULL) ? (ir_rvalue *) constant : (ir_rvalue *) result;
 }
 
-
 /**
  * Dereference a specific component from a scalar, vector, or matrix
  */
@@ -239,7 +275,7 @@ process_array_constructor(exec_list *instructions,
                          YYLTYPE *loc, exec_list *parameters,
                          struct _mesa_glsl_parse_state *state)
 {
-   void *ctx = talloc_parent(state);
+   void *ctx = state;
    /* Array constructors come in two forms: sized and unsized.  Sized array
     * constructors look like 'vec4[2](a, b)', where 'a' and 'b' are vec4
     * variables.  In this case the number of parameters must exactly match the
@@ -279,29 +315,70 @@ process_array_constructor(exec_list *instructions,
 
    if (constructor_type->length == 0) {
       constructor_type =
-        glsl_type::get_array_instance(state,
-                                      constructor_type->element_type(),
+        glsl_type::get_array_instance(constructor_type->element_type(),
                                       parameter_count);
       assert(constructor_type != NULL);
       assert(constructor_type->length == parameter_count);
    }
 
-   ir_function *f = state->symbols->get_function(constructor_type->name);
+   bool all_parameters_are_constant = true;
 
-   /* If the constructor for this type of array does not exist, generate the
-    * prototype and add it to the symbol table.
-    */
-   if (f == NULL) {
-      f = constructor_type->generate_constructor(state->symbols);
+   /* Type cast each parameter and, if possible, fold constants. */
+   foreach_list_safe(n, &actual_parameters) {
+      ir_rvalue *ir = (ir_rvalue *) n;
+      ir_rvalue *result = ir;
+
+      /* Apply implicit conversions (not the scalar constructor rules!) */
+      if (constructor_type->element_type()->is_float()) {
+        const glsl_type *desired_type =
+           glsl_type::get_instance(GLSL_TYPE_FLOAT,
+                                   ir->type->vector_elements,
+                                   ir->type->matrix_columns);
+        result = convert_component(ir, desired_type);
+      }
+
+      if (result->type != constructor_type->element_type()) {
+        _mesa_glsl_error(loc, state, "type error in array constructor: "
+                         "expected: %s, found %s",
+                         constructor_type->element_type()->name,
+                         result->type->name);
+      }
+
+      /* Attempt to convert the parameter to a constant valued expression.
+       * After doing so, track whether or not all the parameters to the
+       * constructor are trivially constant valued expressions.
+       */
+      ir_rvalue *const constant = result->constant_expression_value();
+
+      if (constant != NULL)
+         result = constant;
+      else
+         all_parameters_are_constant = false;
+
+      ir->replace_with(result);
    }
 
-   ir_rvalue *const r =
-      process_call(instructions, f, loc, &actual_parameters, state);
+   if (all_parameters_are_constant) {
+      /* FINISHME: Add support for generating constant arrays. */
+   }
+
+   ir_variable *var = new(ctx) ir_variable(constructor_type, "array_ctor",
+                                          ir_var_temporary);
+   instructions->push_tail(var);
+
+   int i = 0;
+   foreach_list(node, &actual_parameters) {
+      ir_rvalue *rhs = (ir_rvalue *) node;
+      ir_rvalue *lhs = new(ctx) ir_dereference_array(var,
+                                                    new(ctx) ir_constant(i));
+
+      ir_instruction *assignment = new(ctx) ir_assignment(lhs, rhs, NULL);
+      instructions->push_tail(assignment);
 
-   assert(r != NULL);
-   assert(r->type->is_error() || (r->type == constructor_type));
+      i++;
+   }
 
-   return r;
+   return new(ctx) ir_dereference_variable(var);
 }
 
 
@@ -313,7 +390,7 @@ constant_record_constructor(const glsl_type *constructor_type,
                            YYLTYPE *loc, exec_list *parameters,
                            struct _mesa_glsl_parse_state *state)
 {
-   void *ctx = talloc_parent(state);
+   void *ctx = state;
    bool all_parameters_are_constant = true;
 
    exec_node *node = parameters->head;
@@ -476,7 +553,9 @@ emit_inline_vector_constructor(const glsl_type *type,
 {
    assert(!parameters->is_empty());
 
-   ir_variable *var = new(ctx) ir_variable(type, strdup("vec_ctor"));
+   ir_variable *var = new(ctx) ir_variable(type,
+                                          talloc_strdup(ctx, "vec_ctor"),
+                                          ir_var_temporary);
    instructions->push_tail(var);
 
    /* There are two kinds of vector constructors.
@@ -536,14 +615,334 @@ emit_inline_vector_constructor(const glsl_type *type,
 }
 
 
+/**
+ * Generate assignment of a portion of a vector to a portion of a matrix column
+ *
+ * \param src_base  First component of the source to be used in assignment
+ * \param column    Column of destination to be assiged
+ * \param row_base  First component of the destination column to be assigned
+ * \param count     Number of components to be assigned
+ *
+ * \note
+ * \c src_base + \c count must be less than or equal to the number of components
+ * in the source vector.
+ */
+ir_instruction *
+assign_to_matrix_column(ir_variable *var, unsigned column, unsigned row_base,
+                       ir_rvalue *src, unsigned src_base, unsigned count,
+                       TALLOC_CTX *ctx)
+{
+   const unsigned mask[8] = { 0, 1, 2, 3, 0, 0, 0, 0 };
+
+   ir_constant *col_idx = new(ctx) ir_constant(column);
+   ir_rvalue *column_ref = new(ctx) ir_dereference_array(var, col_idx);
+
+   assert(column_ref->type->components() >= (row_base + count));
+   ir_rvalue *lhs = new(ctx) ir_swizzle(column_ref, &mask[row_base], count);
+
+   assert(src->type->components() >= (src_base + count));
+   ir_rvalue *rhs = new(ctx) ir_swizzle(src, &mask[src_base], count);
+
+   return new(ctx) ir_assignment(lhs, rhs, NULL);
+}
+
+
+/**
+ * Generate inline code for a matrix constructor
+ *
+ * The generated constructor code will consist of a temporary variable
+ * declaration of the same type as the constructor.  A sequence of assignments
+ * from constructor parameters to the temporary will follow.
+ *
+ * \return
+ * An \c ir_dereference_variable of the temprorary generated in the constructor
+ * body.
+ */
+ir_rvalue *
+emit_inline_matrix_constructor(const glsl_type *type,
+                              exec_list *instructions,
+                              exec_list *parameters,
+                              void *ctx)
+{
+   assert(!parameters->is_empty());
+
+   ir_variable *var = new(ctx) ir_variable(type,
+                                          talloc_strdup(ctx, "mat_ctor"),
+                                          ir_var_temporary);
+   instructions->push_tail(var);
+
+   /* There are three kinds of matrix constructors.
+    *
+    *  - Construct a matrix from a single scalar by replicating that scalar to
+    *    along the diagonal of the matrix and setting all other components to
+    *    zero.
+    *
+    *  - Construct a matrix from an arbirary combination of vectors and
+    *    scalars.  The components of the constructor parameters are assigned
+    *    to the matrix in colum-major order until the matrix is full.
+    *
+    *  - Construct a matrix from a single matrix.  The source matrix is copied
+    *    to the upper left portion of the constructed matrix, and the remaining
+    *    elements take values from the identity matrix.
+    */
+   ir_rvalue *const first_param = (ir_rvalue *) parameters->head;
+   if (single_scalar_parameter(parameters)) {
+      /* Assign the scalar to the X component of a vec4, and fill the remaining
+       * components with zero.
+       */
+      ir_variable *rhs_var =
+        new(ctx) ir_variable(glsl_type::vec4_type,
+                             talloc_strdup(ctx, "mat_ctor_vec"),
+                             ir_var_temporary);
+      instructions->push_tail(rhs_var);
+
+      ir_constant_data zero;
+      zero.f[0] = 0.0;
+      zero.f[1] = 0.0;
+      zero.f[2] = 0.0;
+      zero.f[3] = 0.0;
+
+      ir_instruction *inst =
+        new(ctx) ir_assignment(new(ctx) ir_dereference_variable(rhs_var),
+                               new(ctx) ir_constant(rhs_var->type, &zero),
+                               NULL);
+      instructions->push_tail(inst);
+
+      ir_rvalue *const rhs_ref = new(ctx) ir_dereference_variable(rhs_var);
+      ir_rvalue *const x_of_rhs = new(ctx) ir_swizzle(rhs_ref, 0, 0, 0, 0, 1);
+
+      inst = new(ctx) ir_assignment(x_of_rhs, first_param, NULL);
+      instructions->push_tail(inst);
+
+      /* Assign the temporary vector to each column of the destination matrix
+       * with a swizzle that puts the X component on the diagonal of the
+       * matrix.  In some cases this may mean that the X component does not
+       * get assigned into the column at all (i.e., when the matrix has more
+       * columns than rows).
+       */
+      static const unsigned rhs_swiz[4][4] = {
+        { 0, 1, 1, 1 },
+        { 1, 0, 1, 1 },
+        { 1, 1, 0, 1 },
+        { 1, 1, 1, 0 }
+      };
+
+      const unsigned cols_to_init = min(type->matrix_columns,
+                                       type->vector_elements);
+      for (unsigned i = 0; i < cols_to_init; i++) {
+        ir_constant *const col_idx = new(ctx) ir_constant(i);
+        ir_rvalue *const col_ref = new(ctx) ir_dereference_array(var, col_idx);
+
+        ir_rvalue *const rhs_ref = new(ctx) ir_dereference_variable(rhs_var);
+        ir_rvalue *const rhs = new(ctx) ir_swizzle(rhs_ref, rhs_swiz[i],
+                                                   type->vector_elements);
+
+        inst = new(ctx) ir_assignment(col_ref, rhs, NULL);
+        instructions->push_tail(inst);
+      }
+
+      for (unsigned i = cols_to_init; i < type->matrix_columns; i++) {
+        ir_constant *const col_idx = new(ctx) ir_constant(i);
+        ir_rvalue *const col_ref = new(ctx) ir_dereference_array(var, col_idx);
+
+        ir_rvalue *const rhs_ref = new(ctx) ir_dereference_variable(rhs_var);
+        ir_rvalue *const rhs = new(ctx) ir_swizzle(rhs_ref, 1, 1, 1, 1,
+                                                   type->vector_elements);
+
+        inst = new(ctx) ir_assignment(col_ref, rhs, NULL);
+        instructions->push_tail(inst);
+      }
+   } else if (first_param->type->is_matrix()) {
+      /* From page 50 (56 of the PDF) of the GLSL 1.50 spec:
+       *
+       *     "If a matrix is constructed from a matrix, then each component
+       *     (column i, row j) in the result that has a corresponding
+       *     component (column i, row j) in the argument will be initialized
+       *     from there. All other components will be initialized to the
+       *     identity matrix. If a matrix argument is given to a matrix
+       *     constructor, it is an error to have any other arguments."
+       */
+      assert(first_param->next->is_tail_sentinal());
+      ir_rvalue *const src_matrix = first_param;
+
+      /* If the source matrix is smaller, pre-initialize the relavent parts of
+       * the destination matrix to the identity matrix.
+       */
+      if ((src_matrix->type->matrix_columns < var->type->matrix_columns)
+         || (src_matrix->type->vector_elements < var->type->vector_elements)) {
+
+        /* If the source matrix has fewer rows, every column of the destination
+         * must be initialized.  Otherwise only the columns in the destination
+         * that do not exist in the source must be initialized.
+         */
+        unsigned col =
+           (src_matrix->type->vector_elements < var->type->vector_elements)
+           ? 0 : src_matrix->type->matrix_columns;
+
+        const glsl_type *const col_type = var->type->column_type();
+        for (/* empty */; col < var->type->matrix_columns; col++) {
+           ir_constant_data ident;
+
+           ident.f[0] = 0.0;
+           ident.f[1] = 0.0;
+           ident.f[2] = 0.0;
+           ident.f[3] = 0.0;
+
+           ident.f[col] = 1.0;
+
+           ir_rvalue *const rhs = new(ctx) ir_constant(col_type, &ident);
+
+           ir_rvalue *const lhs =
+              new(ctx) ir_dereference_array(var, new(ctx) ir_constant(col));
+
+           ir_instruction *inst = new(ctx) ir_assignment(lhs, rhs, NULL);
+           instructions->push_tail(inst);
+        }
+      }
+
+      /* Assign columns from the source matrix to the destination matrix.
+       *
+       * Since the parameter will be used in the RHS of multiple assignments,
+       * generate a temporary and copy the paramter there.
+       */
+      ir_variable *const rhs_var =
+        new(ctx) ir_variable(first_param->type,
+                             talloc_strdup(ctx, "mat_ctor_mat"),
+                             ir_var_temporary);
+      instructions->push_tail(rhs_var);
+
+      ir_dereference *const rhs_var_ref =
+        new(ctx) ir_dereference_variable(rhs_var);
+      ir_instruction *const inst =
+        new(ctx) ir_assignment(rhs_var_ref, first_param, NULL);
+      instructions->push_tail(inst);
+
+
+      const unsigned swiz[4] = { 0, 1, 2, 3 };
+      const unsigned last_col = min(src_matrix->type->matrix_columns,
+                                   var->type->matrix_columns);
+      for (unsigned i = 0; i < last_col; i++) {
+        ir_rvalue *const lhs_col =
+           new(ctx) ir_dereference_array(var, new(ctx) ir_constant(i));
+        ir_rvalue *const rhs_col =
+           new(ctx) ir_dereference_array(rhs_var, new(ctx) ir_constant(i));
+
+        /* If one matrix has columns that are smaller than the columns of the
+         * other matrix, wrap the column access of the larger with a swizzle
+         * so that the LHS and RHS of the assignment have the same size (and
+         * therefore have the same type).
+         *
+         * It would be perfectly valid to unconditionally generate the
+         * swizzles, this this will typically result in a more compact IR tree.
+         */
+        ir_rvalue *lhs;
+        ir_rvalue *rhs;
+        if (lhs_col->type->vector_elements < rhs_col->type->vector_elements) {
+           lhs = lhs_col;
+
+           rhs = new(ctx) ir_swizzle(rhs_col, swiz,
+                                     lhs_col->type->vector_elements);
+        } else if (lhs_col->type->vector_elements
+                   > rhs_col->type->vector_elements) {
+           lhs = new(ctx) ir_swizzle(lhs_col, swiz,
+                                     rhs_col->type->vector_elements);
+           rhs = rhs_col;
+        } else {
+           lhs = lhs_col;
+           rhs = rhs_col;
+        }
+
+        assert(lhs->type == rhs->type);
+
+        ir_instruction *inst = new(ctx) ir_assignment(lhs, rhs, NULL);
+        instructions->push_tail(inst);
+      }
+   } else {
+      const unsigned rows = type->matrix_columns;
+      const unsigned cols = type->vector_elements;
+      unsigned col_idx = 0;
+      unsigned row_idx = 0;
+
+      foreach_list (node, parameters) {
+        ir_rvalue *const rhs = (ir_rvalue *) node;
+        const unsigned components_remaining_this_column = rows - row_idx;
+        unsigned rhs_components = rhs->type->components();
+        unsigned rhs_base = 0;
+
+        /* Since the parameter might be used in the RHS of two assignments,
+         * generate a temporary and copy the paramter there.
+         */
+        ir_variable *rhs_var =
+           new(ctx) ir_variable(rhs->type,
+                                talloc_strdup(ctx, "mat_ctor_vec"),
+                                ir_var_temporary);
+        instructions->push_tail(rhs_var);
+
+        ir_dereference *rhs_var_ref =
+           new(ctx) ir_dereference_variable(rhs_var);
+        ir_instruction *inst = new(ctx) ir_assignment(rhs_var_ref, rhs, NULL);
+        instructions->push_tail(inst);
+
+        /* Assign the current parameter to as many components of the matrix
+         * as it will fill.
+         *
+         * NOTE: A single vector parameter can span two matrix columns.  A
+         * single vec4, for example, can completely fill a mat2.
+         */
+        if (rhs_components >= components_remaining_this_column) {
+           const unsigned count = min(rhs_components,
+                                      components_remaining_this_column);
+
+           rhs_var_ref = new(ctx) ir_dereference_variable(rhs_var);
+
+           ir_instruction *inst = assign_to_matrix_column(var, col_idx,
+                                                          row_idx,
+                                                          rhs_var_ref, 0,
+                                                          count, ctx);
+           instructions->push_tail(inst);
+
+           rhs_base = count;
+
+           col_idx++;
+           row_idx = 0;
+        }
+
+        /* If there is data left in the parameter and components left to be
+         * set in the destination, emit another assignment.  It is possible
+         * that the assignment could be of a vec4 to the last element of the
+         * matrix.  In this case col_idx==cols, but there is still data
+         * left in the source parameter.  Obviously, don't emit an assignment
+         * to data outside the destination matrix.
+         */
+        if ((col_idx < cols) && (rhs_base < rhs_components)) {
+           const unsigned count = rhs_components - rhs_base;
+
+           rhs_var_ref = new(ctx) ir_dereference_variable(rhs_var);
+
+           ir_instruction *inst = assign_to_matrix_column(var, col_idx,
+                                                          row_idx,
+                                                          rhs_var_ref,
+                                                          rhs_base,
+                                                          count, ctx);
+           instructions->push_tail(inst);
+
+           row_idx += count;
+        }
+      }
+   }
+
+   return new(ctx) ir_dereference_variable(var);
+}
+
+
 ir_rvalue *
 ast_function_expression::hir(exec_list *instructions,
                             struct _mesa_glsl_parse_state *state)
 {
-   void *ctx = talloc_parent(state);
+   void *ctx = state;
    /* There are three sorts of function calls.
     *
-    * 1. contstructors - The first subexpression is an ast_type_specifier.
+    * 1. constructors - The first subexpression is an ast_type_specifier.
     * 2. methods - Only the .length() method of array types.
     * 3. functions - Calls to regular old functions.
     *
@@ -585,243 +984,197 @@ ast_function_expression::hir(exec_list *instructions,
        * correct order.  These constructors follow essentially the same type
        * matching rules as functions.
        */
-      if (constructor_type->is_numeric() || constructor_type->is_boolean()) {
-        /* Constructing a numeric type has a couple steps.  First all values
-         * passed to the constructor are broken into individual parameters
-         * and type converted to the base type of the thing being constructed.
-         *
-         * At that point we have some number of values that match the base
-         * type of the thing being constructed.  Now the constructor can be
-         * treated like a function call.  Each numeric type has a small set
-         * of constructor functions.  The set of new parameters will either
-         * match one of those functions or the original constructor is
-         * invalid.
-         */
-        const glsl_type *const base_type = constructor_type->get_base_type();
+      if (!constructor_type->is_numeric() && !constructor_type->is_boolean())
+        return ir_call::get_error_instruction(ctx);
 
-        /* Total number of components of the type being constructed.
-         */
-        const unsigned type_components = constructor_type->components();
+      /* Total number of components of the type being constructed. */
+      const unsigned type_components = constructor_type->components();
 
-        /* Number of components from parameters that have actually been
-         * consumed.  This is used to perform several kinds of error checking.
-         */
-        unsigned components_used = 0;
+      /* Number of components from parameters that have actually been
+       * consumed.  This is used to perform several kinds of error checking.
+       */
+      unsigned components_used = 0;
 
-        unsigned matrix_parameters = 0;
-        unsigned nonmatrix_parameters = 0;
-        exec_list actual_parameters;
+      unsigned matrix_parameters = 0;
+      unsigned nonmatrix_parameters = 0;
+      exec_list actual_parameters;
 
-        bool all_parameters_are_constant = true;
+      foreach_list (n, &this->expressions) {
+        ast_node *ast = exec_node_data(ast_node, n, link);
+        ir_rvalue *result = ast->hir(instructions, state)->as_rvalue();
 
-        /* This handles invalid constructor calls such as 'vec4 v = vec4();'
+        /* From page 50 (page 56 of the PDF) of the GLSL 1.50 spec:
+         *
+         *    "It is an error to provide extra arguments beyond this
+         *    last used argument."
          */
-        if (this->expressions.is_empty()) {
-           _mesa_glsl_error(& loc, state, "too few components to construct "
-                            "`%s'",
+        if (components_used >= type_components) {
+           _mesa_glsl_error(& loc, state, "too many parameters to `%s' "
+                            "constructor",
                             constructor_type->name);
            return ir_call::get_error_instruction(ctx);
         }
 
-        foreach_list (n, &this->expressions) {
-           ast_node *ast = exec_node_data(ast_node, n, link);
-           ir_rvalue *result =
-              ast->hir(instructions, state)->as_rvalue();
-           ir_variable *result_var = NULL;
-
-           /* Attempt to convert the parameter to a constant valued expression.
-            * After doing so, track whether or not all the parameters to the
-            * constructor are trivially constant valued expressions.
-            */
-           ir_rvalue *const constant =
-              result->constant_expression_value();
-
-           if (constant != NULL)
-              result = constant;
-           else
-              all_parameters_are_constant = false;
-
-           /* From page 50 (page 56 of the PDF) of the GLSL 1.50 spec:
-            *
-            *    "It is an error to provide extra arguments beyond this
-            *    last used argument."
-            */
-           if (components_used >= type_components) {
-              _mesa_glsl_error(& loc, state, "too many parameters to `%s' "
-                               "constructor",
-                               constructor_type->name);
-              return ir_call::get_error_instruction(ctx);
-           }
+        if (!result->type->is_numeric() && !result->type->is_boolean()) {
+           _mesa_glsl_error(& loc, state, "cannot construct `%s' from a "
+                            "non-numeric data type",
+                            constructor_type->name);
+           return ir_call::get_error_instruction(ctx);
+        }
 
-           if (!result->type->is_numeric() && !result->type->is_boolean()) {
-              _mesa_glsl_error(& loc, state, "cannot construct `%s' from a "
-                               "non-numeric data type",
-                               constructor_type->name);
-              return ir_call::get_error_instruction(ctx);
-           }
+        /* Count the number of matrix and nonmatrix parameters.  This
+         * is used below to enforce some of the constructor rules.
+         */
+        if (result->type->is_matrix())
+           matrix_parameters++;
+        else
+           nonmatrix_parameters++;
 
-           /* Count the number of matrix and nonmatrix parameters.  This
-            * is used below to enforce some of the constructor rules.
-            */
-           if (result->type->is_matrix())
-              matrix_parameters++;
-           else
-              nonmatrix_parameters++;
-
-           /* We can't use the same instruction node in the multiple
-            * swizzle dereferences that happen, so assign it to a
-            * variable and deref that.  Plus it saves computation for
-            * complicated expressions and handles
-            * glsl-vs-constructor-call.shader_test.
-            */
-           if (result->type->components() >= 1 && !result->as_constant()) {
-              result_var = new(ctx) ir_variable(result->type,
-                                                "constructor_tmp");
-              ir_dereference_variable *lhs;
-
-              lhs = new(ctx) ir_dereference_variable(result_var);
-              instructions->push_tail(new(ctx) ir_assignment(lhs,
-                                                             result, NULL));
-           }
+        actual_parameters.push_tail(result);
+        components_used += result->type->components();
+      }
 
-           /* Process each of the components of the parameter.  Dereference
-            * each component individually, perform any type conversions, and
-            * add it to the parameter list for the constructor.
-            */
-           for (unsigned i = 0; i < result->type->components(); i++) {
-              if (components_used >= type_components)
-                 break;
-
-              ir_rvalue *component;
-
-              if (result_var) {
-                 ir_dereference *d = new(ctx) ir_dereference_variable(result_var);
-                 component = dereference_component(d, i);
-              } else {
-                 component = dereference_component(result, i);
-              }
-              component = convert_component(component, base_type);
-
-              /* All cases that could result in component->type being the
-               * error type should have already been caught above.
-               */
-              assert(component->type == base_type);
+      /* From page 28 (page 34 of the PDF) of the GLSL 1.10 spec:
+       *
+       *    "It is an error to construct matrices from other matrices. This
+       *    is reserved for future use."
+       */
+      if ((state->language_version <= 110) && (matrix_parameters > 0)
+         && constructor_type->is_matrix()) {
+        _mesa_glsl_error(& loc, state, "cannot construct `%s' from a "
+                         "matrix in GLSL 1.10",
+                         constructor_type->name);
+        return ir_call::get_error_instruction(ctx);
+      }
 
-              if (component->as_constant() == NULL)
-                 all_parameters_are_constant = false;
+      /* From page 50 (page 56 of the PDF) of the GLSL 1.50 spec:
+       *
+       *    "If a matrix argument is given to a matrix constructor, it is
+       *    an error to have any other arguments."
+       */
+      if ((matrix_parameters > 0)
+         && ((matrix_parameters + nonmatrix_parameters) > 1)
+         && constructor_type->is_matrix()) {
+        _mesa_glsl_error(& loc, state, "for matrix `%s' constructor, "
+                         "matrix must be only parameter",
+                         constructor_type->name);
+        return ir_call::get_error_instruction(ctx);
+      }
 
-              /* Don't actually generate constructor calls for scalars.
-               * Instead, do the usual component selection and conversion,
-               * and return the single component.
-               */
-              if (constructor_type->is_scalar())
-                 return component;
+      /* From page 28 (page 34 of the PDF) of the GLSL 1.10 spec:
+       *
+       *    "In these cases, there must be enough components provided in the
+       *    arguments to provide an initializer for every component in the
+       *    constructed value."
+       */
+      if ((components_used < type_components) && (components_used != 1)) {
+        _mesa_glsl_error(& loc, state, "too few components to construct "
+                         "`%s'",
+                         constructor_type->name);
+        return ir_call::get_error_instruction(ctx);
+      }
 
-              actual_parameters.push_tail(component);
-              components_used++;
+      /* Later, we cast each parameter to the same base type as the
+       * constructor.  Since there are no non-floating point matrices, we
+       * need to break them up into a series of column vectors.
+       */
+      if (constructor_type->base_type != GLSL_TYPE_FLOAT) {
+        foreach_list_safe(n, &actual_parameters) {
+           ir_rvalue *matrix = (ir_rvalue *) n;
+
+           if (!matrix->type->is_matrix())
+              continue;
+
+           /* Create a temporary containing the matrix. */
+           ir_variable *var = new(ctx) ir_variable(matrix->type, "matrix_tmp",
+                                                   ir_var_temporary);
+           instructions->push_tail(var);
+           instructions->push_tail(new(ctx) ir_assignment(new(ctx)
+              ir_dereference_variable(var), matrix, NULL));
+           var->constant_value = matrix->constant_expression_value();
+
+           /* Replace the matrix with dereferences of its columns. */
+           for (int i = 0; i < matrix->type->matrix_columns; i++) {
+              matrix->insert_before(new (ctx) ir_dereference_array(var,
+                 new(ctx) ir_constant(i)));
            }
+           matrix->remove();
         }
+      }
 
-        /* From page 28 (page 34 of the PDF) of the GLSL 1.10 spec:
-         *
-         *    "It is an error to construct matrices from other matrices. This
-         *    is reserved for future use."
-         */
-        if ((state->language_version <= 110) && (matrix_parameters > 0)
-            && constructor_type->is_matrix()) {
-           _mesa_glsl_error(& loc, state, "cannot construct `%s' from a "
-                            "matrix in GLSL 1.10",
-                            constructor_type->name);
-           return ir_call::get_error_instruction(ctx);
-        }
+      bool all_parameters_are_constant = true;
 
-        /* From page 50 (page 56 of the PDF) of the GLSL 1.50 spec:
-         *
-         *    "If a matrix argument is given to a matrix constructor, it is
-         *    an error to have any other arguments."
-         */
-        if ((matrix_parameters > 0)
-            && ((matrix_parameters + nonmatrix_parameters) > 1)
-            && constructor_type->is_matrix()) {
-           _mesa_glsl_error(& loc, state, "for matrix `%s' constructor, "
-                            "matrix must be only parameter",
-                            constructor_type->name);
-           return ir_call::get_error_instruction(ctx);
-        }
+      /* Type cast each parameter and, if possible, fold constants.*/
+      foreach_list_safe(n, &actual_parameters) {
+        ir_rvalue *ir = (ir_rvalue *) n;
 
-        /* From page 28 (page 34 of the PDF) of the GLSL 1.10 spec:
-         *
-         *    "In these cases, there must be enough components provided in the
-         *    arguments to provide an initializer for every component in the
-         *    constructed value."
+        const glsl_type *desired_type =
+           glsl_type::get_instance(constructor_type->base_type,
+                                   ir->type->vector_elements,
+                                   ir->type->matrix_columns);
+        ir_rvalue *result = convert_component(ir, desired_type);
+
+        /* Attempt to convert the parameter to a constant valued expression.
+         * After doing so, track whether or not all the parameters to the
+         * constructor are trivially constant valued expressions.
          */
-        if ((components_used < type_components) && (components_used != 1)) {
-           _mesa_glsl_error(& loc, state, "too few components to construct "
-                            "`%s'",
-                            constructor_type->name);
-           return ir_call::get_error_instruction(ctx);
-        }
+        ir_rvalue *const constant = result->constant_expression_value();
 
-        ir_function *f = state->symbols->get_function(constructor_type->name);
-        if (f == NULL) {
-           _mesa_glsl_error(& loc, state, "no constructor for type `%s'",
-                            constructor_type->name);
-           return ir_call::get_error_instruction(ctx);
-        }
+        if (constant != NULL)
+           result = constant;
+        else
+           all_parameters_are_constant = false;
 
-        const ir_function_signature *sig =
-           f->matching_signature(& actual_parameters);
-        if (sig != NULL) {
-           /* If all of the parameters are trivially constant, create a
-            * constant representing the complete collection of parameters.
-            */
-           if (all_parameters_are_constant) {
-              if (components_used >= type_components)
-                 return new(ctx) ir_constant(sig->return_type,
-                                             & actual_parameters);
-
-              assert(sig->return_type->is_vector()
-                     || sig->return_type->is_matrix());
-
-              /* Constructors with exactly one component are special for
-               * vectors and matrices.  For vectors it causes all elements of
-               * the vector to be filled with the value.  For matrices it
-               * causes the matrix to be filled with 0 and the diagonal to be
-               * filled with the value.
-               */
-              ir_constant_data data;
-              ir_constant *const initializer =
-                 (ir_constant *) actual_parameters.head;
-              if (sig->return_type->is_matrix())
-                 generate_constructor_matrix(sig->return_type, initializer,
-                                             &data);
-              else
-                 generate_constructor_vector(sig->return_type, initializer,
-                                             &data);
-
-              return new(ctx) ir_constant(sig->return_type, &data);
-           } else if (constructor_type->is_vector()) {
-              return emit_inline_vector_constructor(constructor_type,
-                                                    instructions,
-                                                    &actual_parameters,
-                                                    ctx);
-           } else {
-              assert(constructor_type->is_matrix());
-              return new(ctx) ir_call(sig, & actual_parameters);
-           }
-        } else {
-           /* FINISHME: Log a better error message here.  G++ will show the
-            * FINSIHME: types of the actual parameters and the set of
-            * FINSIHME: candidate functions.  A different error should also be
-            * FINSIHME: logged when multiple functions match.
-            */
-           _mesa_glsl_error(& loc, state, "no matching constructor for `%s'",
-                            constructor_type->name);
-           return ir_call::get_error_instruction(ctx);
+        if (result != ir) {
+           ir->replace_with(result);
         }
       }
 
-      return ir_call::get_error_instruction(ctx);
+      /* If all of the parameters are trivially constant, create a
+       * constant representing the complete collection of parameters.
+       */
+      if (all_parameters_are_constant) {
+        if (components_used >= type_components)
+           return new(ctx) ir_constant(constructor_type,
+                                       & actual_parameters);
+
+        /* The above case must handle all scalar constructors.
+         */
+        assert(constructor_type->is_vector()
+               || constructor_type->is_matrix());
+
+        /* Constructors with exactly one component are special for
+         * vectors and matrices.  For vectors it causes all elements of
+         * the vector to be filled with the value.  For matrices it
+         * causes the matrix to be filled with 0 and the diagonal to be
+         * filled with the value.
+         */
+        ir_constant_data data;
+        ir_constant *const initializer =
+           (ir_constant *) actual_parameters.head;
+        if (constructor_type->is_matrix())
+           generate_constructor_matrix(constructor_type, initializer,
+                                       &data);
+        else
+           generate_constructor_vector(constructor_type, initializer,
+                                       &data);
+
+        return new(ctx) ir_constant(constructor_type, &data);
+      } else if (constructor_type->is_scalar()) {
+        return dereference_component((ir_rvalue *) actual_parameters.head,
+                                     0);
+      } else if (constructor_type->is_vector()) {
+        return emit_inline_vector_constructor(constructor_type,
+                                              instructions,
+                                              &actual_parameters,
+                                              ctx);
+      } else {
+        assert(constructor_type->is_matrix());
+        return emit_inline_matrix_constructor(constructor_type,
+                                              instructions,
+                                              &actual_parameters,
+                                              ctx);
+      }
    } else {
       const ast_expression *id = subexpressions[0];
       YYLTYPE loc = id->get_location();