From 2747f6a1f9d27826346d330cceef44b800267bcb Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Sun, 25 May 2014 01:08:50 -0700 Subject: [PATCH] i965/sf: Drop unnecessary push/pop in do_twoside_color. None of the assembly emitters called between push and pop actually change the state. So, we can drop these. Signed-off-by: Kenneth Graunke Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/brw_sf_emit.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_sf_emit.c b/src/mesa/drivers/dri/i965/brw_sf_emit.c index 96be91a14d6..475d4898521 100644 --- a/src/mesa/drivers/dri/i965/brw_sf_emit.c +++ b/src/mesa/drivers/dri/i965/brw_sf_emit.c @@ -137,7 +137,6 @@ static void do_twoside_color( struct brw_sf_compile *c ) * we run with NoMask, so it's not an option and we can use * BRW_EXECUTE_1 for all comparisions. */ - brw_push_insn_state(p); brw_CMP(p, vec4(brw_null_reg()), backface_conditional, c->det, brw_imm_f(0)); brw_IF(p, BRW_EXECUTE_4); { @@ -148,7 +147,6 @@ static void do_twoside_color( struct brw_sf_compile *c ) } } brw_ENDIF(p); - brw_pop_insn_state(p); } -- 2.30.2