glsl: Minor clean-up in validate_assignment
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 10 Dec 2010 23:47:11 +0000 (15:47 -0800)
committerIan Romanick <ian.d.romanick@intel.com>
Sat, 11 Dec 2010 01:52:35 +0000 (17:52 -0800)
This code has been changed around a lot, and there were some temporary
variables left around from previous versions.

src/glsl/ast_to_hir.cpp

index 1f4972cfca20a800e7513ebdc9355e7761697b7d..75ba3a9bafa704d428b34b72d7b0d0017ff2563e 100644 (file)
@@ -598,17 +598,15 @@ ir_rvalue *
 validate_assignment(struct _mesa_glsl_parse_state *state,
                    const glsl_type *lhs_type, ir_rvalue *rhs)
 {
-   const glsl_type *rhs_type = rhs->type;
-
    /* If there is already some error in the RHS, just return it.  Anything
     * else will lead to an avalanche of error message back to the user.
     */
-   if (rhs_type->is_error())
+   if (rhs->type->is_error())
       return rhs;
 
    /* If the types are identical, the assignment can trivially proceed.
     */
-   if (rhs_type == lhs_type)
+   if (rhs->type == lhs_type)
       return rhs;
 
    /* If the array element types are the same and the size of the LHS is zero,
@@ -625,8 +623,7 @@ validate_assignment(struct _mesa_glsl_parse_state *state,
 
    /* Check for implicit conversion in GLSL 1.20 */
    if (apply_implicit_conversion(lhs_type, rhs, state)) {
-      rhs_type = rhs->type;
-      if (rhs_type == lhs_type)
+      if (rhs->type == lhs_type)
         return rhs;
    }