i965/vec4: Don't trim writemasks of texture instructions.
authorMatt Turner <mattst88@gmail.com>
Fri, 28 Mar 2014 23:45:17 +0000 (16:45 -0700)
committerMatt Turner <mattst88@gmail.com>
Mon, 31 Mar 2014 17:24:10 +0000 (10:24 -0700)
It was my understanding that the writemask works in SIMD4x2 mode for
texturing instructions and doesn't require a message header. Some bit of
this logic must be wrong, so disable it until it's understood.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76617
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_vec4.cpp

index 4ae6020..32a3892 100644 (file)
@@ -351,8 +351,10 @@ try_eliminate_instruction(vec4_instruction *inst, int new_writemask)
       case VS_OPCODE_PULL_CONSTANT_LOAD_GEN7:
          break;
       default:
-         inst->dst.writemask = new_writemask;
-         return true;
+         if (!inst->is_tex()) {
+            inst->dst.writemask = new_writemask;
+            return true;
+         }
       }
    }