Mesa (master): glsl: Function signatures cannot have NULL return type

[Ian Romanick]

Module: Mesa
Branch: master
Commit: 2df56b002dcc5d7e91515bd0ca741677f0172b38
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2df56b002dcc5d7e91515bd0ca741677f0172b38

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Mon Mar  7 15:08:22 2011 -0800

glsl: Function signatures cannot have NULL return type

The return type can be void, and this is the case where a `_ret_val'
variable should not be declared.

---

 src/glsl/ir_validate.cpp           |    6 ++++++
 src/glsl/opt_function_inlining.cpp |    2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/src/glsl/ir_validate.cpp b/src/glsl/ir_validate.cpp
index 44d7549..5e491db 100644
--- a/src/glsl/ir_validate.cpp
+++ b/src/glsl/ir_validate.cpp
@@ -198,6 +198,12 @@ ir_validate::visit_enter(ir_function_signature *ir)
       abort();
    }
 
+   if (ir->return_type == NULL) {
+      printf("Function signature %p for function %s has NULL return type.\n",
+	     ir, ir->function_name());
+      abort();
+   }
+
    this->validate_ir(ir, this->data);
 
    return visit_continue;
diff --git a/src/glsl/opt_function_inlining.cpp b/src/glsl/opt_function_inlining.cpp
index 2e7831d..064089a 100644
--- a/src/glsl/opt_function_inlining.cpp
+++ b/src/glsl/opt_function_inlining.cpp
@@ -126,7 +126,7 @@ ir_call::generate_inline(ir_instruction *next_ir)
    parameters = new ir_variable *[num_parameters];
 
    /* Generate storage for the return value. */
-   if (this->callee->return_type) {
+   if (!this->callee->return_type->is_void()) {
       retval = new(ctx) ir_variable(this->callee->return_type, "_ret_val",
 				    ir_var_auto);
       next_ir->insert_before(retval);