From: Ian Romanick <ian.d.romanick@intel.com>
Date: Thu, 1 Apr 2010 00:39:10 +0000 (-0700)
Subject: Minor cleanups in ast_function_definition::hir
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=25711a85c22bed305c9b52b89feb9c600d1892df;p=mesa.git

Minor cleanups in ast_function_definition::hir
---

diff --git a/ast_to_hir.cpp b/ast_to_hir.cpp
index e6031ae0046..627d9321821 100644
--- a/ast_to_hir.cpp
+++ b/ast_to_hir.cpp
@@ -1513,7 +1513,7 @@ ast_function_definition::hir(exec_list *instructions,
 
    /* Verify the return type of main() */
    if (strcmp(name, "main") == 0) {
-      if (return_type != glsl_type::get_instance(GLSL_TYPE_VOID, 0, 0)) {
+      if (! return_type->is_void()) {
 	 YYLTYPE loc = this->get_location();
 
 	 _mesa_glsl_error(& loc, state, "main() must return void");
@@ -1551,7 +1551,6 @@ ast_function_definition::hir(exec_list *instructions,
    ast_function_parameters_to_hir(& this->prototype->parameters,
 				  & signature->parameters,
 				  state);
-   /* FINISHME: Set signature->return_type */
 
    label = new ir_label(name);
    if (signature->definition == NULL) {
@@ -1566,9 +1565,9 @@ ast_function_definition::hir(exec_list *instructions,
     */
    state->symbols->push_scope();
    foreach_iter(exec_list_iterator, iter, parameters) {
-      ir_variable *const var = (ir_variable *) iter.get();
+      ir_variable *const var = ((ir_instruction *) iter.get())->as_variable();
 
-      assert(((ir_instruction *) var)->as_variable() != NULL);
+      assert(var != NULL);
 
       iter.remove();
       instructions->push_tail(var);