From: Kenneth Graunke <kenneth@whitecape.org>
Date: Wed, 28 May 2014 07:02:18 +0000 (-0700)
Subject: i965: Drop unnecessary brw_set_conditionalmod() before brw_CMP().
X-Git-Tag: upstream/10.3~1744
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0985da542366cb6be8dc287db4a74c828a7e8fa4;p=platform%2Fupstream%2Fmesa.git

i965: Drop unnecessary brw_set_conditionalmod() before brw_CMP().

brw_CMP already takes a conditional modifier as a parameter, and sets it
accordingly.  brw_set_conditionalmod() also makes everything after the
next instruction predicated, but we don't need that: we always emit an
IF instruction after load_clip_distance(), and that's already
predicated.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
---

diff --git a/src/mesa/drivers/dri/i965/brw_clip_tri.c b/src/mesa/drivers/dri/i965/brw_clip_tri.c
index 9edfa2e..289f4cc 100644
--- a/src/mesa/drivers/dri/i965/brw_clip_tri.c
+++ b/src/mesa/drivers/dri/i965/brw_clip_tri.c
@@ -252,7 +252,6 @@ load_clip_distance(struct brw_clip_compile *c, struct brw_indirect vtx,
    }
    brw_ENDIF(p);
 
-   brw_set_conditionalmod(p, cond);
    brw_CMP(p, brw_null_reg(), cond, vec1(dst), brw_imm_f(0.0f));
 }