From 2335153ff2fae01d6294876a86d3eab59c6c4236 Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Wed, 7 Jan 2015 12:01:43 -0800 Subject: [PATCH] i965: Remove now unnecessary Gen8 CMP destination type override. Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index 57161e282dc..308b305458e 100644 --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c @@ -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); -- 2.30.2