i965/sf: Drop unnecessary push/pop in do_twoside_color.
authorKenneth Graunke <kenneth@whitecape.org>
Sun, 25 May 2014 08:08:50 +0000 (01:08 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 27 May 2014 20:46:01 +0000 (13:46 -0700)
None of the assembly emitters called between push and pop actually
change the state.  So, we can drop these.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_sf_emit.c

index 96be91a14d62dc734a12fa79eb591ecd044b6ee5..475d4898521c488549bba8861363afbce950b733 100644 (file)
@@ -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);
 }