Return the rvalue of a variable decl to fix while (bool b = condition) {}

diff --git a/ast_to_hir.cpp b/ast_to_hir.cpp
index aa9a3a1a04d5be2f3ceb0343ccf55874af9bd13a..4e1c81917719d5d1f5dfe9969143b2150e2b04db 100644 (file)
--- a/ast_to_hir.cpp
+++ b/ast_to_hir.cpp
@@ -1358,7 +1358,7 @@ ast_declarator_list::hir(exec_list *instructions,
    struct simple_node *ptr;
    const struct glsl_type *decl_type;
    const char *type_name = NULL;
-
+   ir_rvalue *result = NULL;
 
    /* FINISHME: Handle vertex shader "invariant" declarations that do not
     * FINISHME: include a type.  These re-declare built-in variables to be
@@ -1619,8 +1619,8 @@ ast_declarator_list::hir(exec_list *instructions,
            bool temp = var->read_only;
            if (this->type->qualifier.constant)
               var->read_only = false;
-           (void) do_assignment(instructions, state, lhs, rhs,
-                                this->get_location());
+           result = do_assignment(instructions, state, lhs, rhs,
+                                  this->get_location());
            var->read_only = temp;
         }
       }
@@ -1650,9 +1650,17 @@ ast_declarator_list::hir(exec_list *instructions,
       assert(added_variable);
    }
 
-   /* Variable declarations do not have r-values.
+
+   /* Generally, variable declarations do not have r-values.  However,
+    * one is used for the declaration in
+    *
+    * while (bool b = some_condition()) {
+    *   ...
+    * }
+    *
+    * so we return the rvalue from the last seen declaration here.
     */
-   return NULL;
+   return result;
 }