From: Eric Anholt <eric@anholt.net>
Date: Wed, 16 Mar 2016 01:58:43 +0000 (-0700)
Subject: vc4: Don't make a temporary for setting flags.
X-Git-Tag: upstream/17.1.0~11885
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7c9fc439150188612c7fe595cbe0180fcea3e705;p=platform%2Fupstream%2Fmesa.git

vc4: Don't make a temporary for setting flags.

The register allocator doesn't really do anything about the temp, so it
doesn't seem like it should matter.  However, the scheduler would think
that a new def is being created.

This doesn't change anything yet, but it avoids a bunch of regressions in
the next commit.
---

diff --git a/src/gallium/drivers/vc4/vc4_qir.c b/src/gallium/drivers/vc4/vc4_qir.c
index fd1192f..125a9525 100644
--- a/src/gallium/drivers/vc4/vc4_qir.c
+++ b/src/gallium/drivers/vc4/vc4_qir.c
@@ -495,7 +495,8 @@ qir_SF(struct vc4_compile *c, struct qreg src)
             !c->defs[src.index] ||
             last_inst != c->defs[src.index] ||
             qir_is_multi_instruction(last_inst)) {
-                src = qir_MOV(c, src);
+                struct qreg null = { QFILE_NULL, 0 };
+                last_inst = qir_MOV_dest(c, null, src);
                 last_inst = (struct qinst *)c->instructions.prev;
         }
         last_inst->sf = true;