Fix error handling of logic operators.
authorEric Anholt <eric@anholt.net>
Thu, 1 Apr 2010 03:05:32 +0000 (17:05 -1000)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 2 Apr 2010 18:22:41 +0000 (11:22 -0700)
They were always throwing a type error because type wasn't being set.

ast_to_hir.cpp

index 5d7c073ca906efb82c0260d87dce86ae8711240e..79e04ee7960bbabfdbc76bc5d4dbbaeec3c4df5a 100644 (file)
@@ -736,6 +736,7 @@ ast_expression::hir(exec_list *instructions,
 
         _mesa_glsl_error(& loc, state, "LHS of `%s' must be scalar boolean",
                          operator_string(this->oper));
+        error_emitted = true;
       }
 
       if (!op[1]->type->is_boolean() || !op[1]->type->is_scalar()) {
@@ -743,10 +744,12 @@ ast_expression::hir(exec_list *instructions,
 
         _mesa_glsl_error(& loc, state, "RHS of `%s' must be scalar boolean",
                          operator_string(this->oper));
+        error_emitted = true;
       }
 
       result = new ir_expression(operations[this->oper], glsl_type::bool_type,
                                 op[0], op[1]);
+      type = glsl_type::bool_type;
       break;
 
    case ast_logic_not:
@@ -757,10 +760,12 @@ ast_expression::hir(exec_list *instructions,
 
         _mesa_glsl_error(& loc, state,
                          "operand of `!' must be scalar boolean");
+        error_emitted = true;
       }
 
       result = new ir_expression(operations[this->oper], glsl_type::bool_type,
                                 op[0], NULL);
+      type = glsl_type::bool_type;
       break;
 
    case ast_mul_assign: