From: Ian Romanick <ian.d.romanick@intel.com>
Date: Wed, 10 Mar 2010 00:38:02 +0000 (-0800)
Subject: Add assignment side-effect to the instruction stream
X-Git-Tag: 062012170305~10660^2~625^2~662
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6652af36fe8994b1621d882fcc230d320908a2a3;p=profile%2Fivi%2Fmesa.git

Add assignment side-effect to the instruction stream

The actual assignment is a side-effect of the assignment expression.
Add it to the instruction stream and return the LHS of the assignment
as its rvalue.
---

diff --git a/ast_to_hir.cpp b/ast_to_hir.cpp
index 5811d73..faa13ab 100644
--- a/ast_to_hir.cpp
+++ b/ast_to_hir.cpp
@@ -398,7 +398,7 @@ ast_expression::hir(exec_list *instructions,
    make_empty_list(& op_list);
 
    switch (this->oper) {
-   case ast_assign:
+   case ast_assign: {
       op[0] = this->subexpressions[0]->hir(instructions, state);
       op[1] = this->subexpressions[1]->hir(instructions, state);
 
@@ -437,8 +437,12 @@ ast_expression::hir(exec_list *instructions,
       /* FINISHME: Check that the LHS and RHS have matching types. */
       /* FINISHME: For GLSL 1.10, check that the types are not arrays. */
 
-      result = new ir_assignment(op[0], op[1], NULL);
+      ir_instruction *tmp = new ir_assignment(op[0], op[1], NULL);
+      instructions->push_tail(tmp);
+
+      result = op[0];
       break;
+   }
 
    case ast_plus:
       op[0] = this->subexpressions[0]->hir(instructions, state);