From: Jakob Bornecrantz <jakob@vmware.com>
Date: Thu, 7 Apr 2011 13:07:20 +0000 (+0100)
Subject: svga: Only emit Z depth work if it is actually read
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;ds=sidebyside;h=4f17830b3dda5a1727a3c87897e73b56b37613a6;hp=3b32e51cb1e0cb3feba01f5612214bac20c5079b;p=mesa.git

svga: Only emit Z depth work if it is actually read
---

diff --git a/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c b/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
index 1c385cc7e81..3b70a3a7635 100644
--- a/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
+++ b/src/gallium/drivers/svga/svga_tgsi_decl_sm30.c
@@ -121,29 +121,35 @@ static boolean ps30_input( struct svga_shader_emitter *emit,
    SVGA3dShaderDestToken reg;
 
    if (semantic.Name == TGSI_SEMANTIC_POSITION) {
+
       emit->ps_true_pos = src_register( SVGA3DREG_MISCTYPE,
                                         SVGA3DMISCREG_POSITION );
-
       emit->ps_true_pos.base.swizzle = TRANSLATE_SWIZZLE( TGSI_SWIZZLE_X,
                                                           TGSI_SWIZZLE_Y,
                                                           TGSI_SWIZZLE_Y,
                                                           TGSI_SWIZZLE_Y );
+      reg = writemask( dst(emit->ps_true_pos),
+                       TGSI_WRITEMASK_XY );
+      emit->ps_reads_pos = TRUE;
 
-      emit->ps_temp_pos = dst_register( SVGA3DREG_TEMP,
-                                        emit->nr_hw_temp );
-      emit->ps_depth_pos = src_register( SVGA3DREG_INPUT, emit->ps30_input_count++ );
-
-      emit->input_map[idx] = src_register( SVGA3DREG_TEMP,
+      if (emit->info.reads_z) {
+         emit->ps_temp_pos = dst_register( SVGA3DREG_TEMP,
                                            emit->nr_hw_temp );
-      emit->nr_hw_temp++;
+         emit->ps_depth_pos = src_register( SVGA3DREG_INPUT,
+                                            emit->ps30_input_count++ );
 
-      reg = writemask( dst(emit->ps_true_pos),
-                       TGSI_WRITEMASK_XY );
+         emit->input_map[idx] = src_register( SVGA3DREG_TEMP,
+                                              emit->nr_hw_temp );
+         emit->nr_hw_temp++;
+
+         if (!emit_decl( emit, dst( emit->ps_depth_pos ),
+                         SVGA3D_DECLUSAGE_TEXCOORD, 1 ))
+            return FALSE;
+      }
+      else {
+         emit->input_map[idx] = emit->ps_true_pos;
+      }
 
-      emit->ps_reads_pos = TRUE;
-      if (!emit_decl( emit, dst(emit->ps_depth_pos),
-                      SVGA3D_DECLUSAGE_TEXCOORD, 1 ))
-         return FALSE;
       return emit_decl( emit, reg, 0, 0 );
    }
    else if (emit->key.fkey.light_twoside &&
diff --git a/src/gallium/drivers/svga/svga_tgsi_insn.c b/src/gallium/drivers/svga/svga_tgsi_insn.c
index ec1906eb619..e6ec5966eaa 100644
--- a/src/gallium/drivers/svga/svga_tgsi_insn.c
+++ b/src/gallium/drivers/svga/svga_tgsi_insn.c
@@ -2550,7 +2550,7 @@ static boolean emit_ps_preamble( struct svga_shader_emitter *emit )
                return FALSE;
          }
       }
-   } else if (emit->ps_reads_pos) {
+   } else if (emit->ps_reads_pos && emit->info.reads_z) {
       /*
        * Assemble the position from various bits of inputs. Depth and W are
        * passed in a texcoord this is due to D3D's vPos not hold Z or W.
@@ -2579,7 +2579,7 @@ static boolean emit_ps_preamble( struct svga_shader_emitter *emit )
                        scalar( src(emit->ps_temp_pos), TGSI_SWIZZLE_W ) ))
          return FALSE;
    }
-   
+
    return TRUE;
 }