i965: Remove now unnecessary Gen8 CMP destination type override.
authorMatt Turner <mattst88@gmail.com>
Wed, 7 Jan 2015 20:01:43 +0000 (12:01 -0800)
committerMatt Turner <mattst88@gmail.com>
Wed, 4 Feb 2015 20:14:35 +0000 (12:14 -0800)
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_eu_emit.c

index 57161e282dc1142fa5d5d1087fbb601b069d9b80..308b305458e9ac244e79d90c63f6db5d6defbdd3 100644 (file)
@@ -1849,14 +1849,6 @@ void brw_CMP(struct brw_compile *p,
    struct brw_context *brw = p->brw;
    brw_inst *insn = next_insn(p, BRW_OPCODE_CMP);
 
-   if (brw->gen >= 8) {
-      /* The CMP instruction appears to behave erratically for floating point
-       * sources unless the destination type is also float.  Overriding it to
-       * match src0 makes it work in all cases.
-       */
-      dest.type = src0.type;
-   }
-
    brw_inst_set_cond_modifier(brw, insn, conditional);
    brw_set_dest(p, insn, dest);
    brw_set_src0(p, insn, src0);