From: Connor Abbott <cwabbott0@gmail.com>
Date: Wed, 22 Jul 2015 02:54:23 +0000 (-0700)
Subject: nir/cf: handle phi nodes better in split_block_beginning()
X-Git-Tag: upstream/17.1.0~16609
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=788d45cb478d6285fe6811c87b4f1db1daded6d9;p=platform%2Fupstream%2Fmesa.git

nir/cf: handle phi nodes better in split_block_beginning()

Signed-off-by: Connor Abbott <connor.w.abbott@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
---

diff --git a/src/glsl/nir/nir_control_flow.c b/src/glsl/nir/nir_control_flow.c
index 205b608..b416a58 100644
--- a/src/glsl/nir/nir_control_flow.c
+++ b/src/glsl/nir/nir_control_flow.c
@@ -210,6 +210,19 @@ split_block_beginning(nir_block *block)
       link_blocks(pred, new_block, NULL);
    }
 
+   /* Any phi nodes must stay part of the new block, or else their
+    * sourcse will be messed up. This will reverse the order of the phi's, but
+    * order shouldn't matter.
+    */
+   nir_foreach_instr_safe(block, instr) {
+      if (instr->type != nir_instr_type_phi)
+         break;
+
+      exec_node_remove(&instr->node);
+      instr->block = new_block;
+      exec_list_push_head(&new_block->instr_list, &instr->node);
+   }
+
    return new_block;
 }