vc4: Eliminate unused texture instructions.
authorEric Anholt <eric@anholt.net>
Thu, 9 Oct 2014 12:42:14 +0000 (14:42 +0200)
committerEric Anholt <eric@anholt.net>
Thu, 9 Oct 2014 19:47:06 +0000 (21:47 +0200)
src/gallium/drivers/vc4/vc4_opt_dead_code.c

index 0c273ff..f08818a 100644 (file)
@@ -52,6 +52,8 @@ qir_opt_dead_code(struct vc4_compile *c)
         bool progress = false;
         bool *used = calloc(c->num_temps, sizeof(bool));
         bool sf_used = false;
+        /* Whether we're eliminating texture setup currently. */
+        bool dce_tex = false;
 
         struct simple_node *node, *t;
         for (node = c->instructions.prev, t = node->prev;
@@ -61,7 +63,13 @@ qir_opt_dead_code(struct vc4_compile *c)
 
                 if (inst->dst.file == QFILE_TEMP &&
                     !used[inst->dst.index] &&
-                    !qir_has_side_effects(inst)) {
+                    (!qir_has_side_effects(inst) ||
+                     inst->op == QOP_TEX_RESULT)) {
+                        if (inst->op == QOP_TEX_RESULT) {
+                                dce_tex = true;
+                                c->num_texture_samples--;
+                        }
+
                         dce(c, inst);
                         progress = true;
                         continue;
@@ -78,6 +86,18 @@ qir_opt_dead_code(struct vc4_compile *c)
                         sf_used = false;
                 }
 
+                if (inst->op == QOP_TEX_RESULT)
+                        dce_tex = false;
+
+                if (dce_tex && (inst->op == QOP_TEX_S ||
+                                inst->op == QOP_TEX_T ||
+                                inst->op == QOP_TEX_R ||
+                                inst->op == QOP_TEX_B)) {
+                        dce(c, inst);
+                        progress = true;
+                        continue;
+                }
+
                 for (int i = 0; i < qir_get_op_nsrc(inst->op); i++) {
                         if (inst->src[i].file == QFILE_TEMP)
                                 used[inst->src[i].index] = true;