From 0985da542366cb6be8dc287db4a74c828a7e8fa4 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Wed, 28 May 2014 00:02:18 -0700 Subject: [PATCH] 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 Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/brw_clip_tri.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_clip_tri.c b/src/mesa/drivers/dri/i965/brw_clip_tri.c index 9edfa2e176c..289f4ccd82a 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)); } -- 2.30.2