i965: Change GEN_GEN < 7 to GEN_GEN == 6 in 3DSTATE_VS code.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 10 May 2017 19:41:38 +0000 (12:41 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 11 May 2017 23:34:04 +0000 (16:34 -0700)
This whole code is surrounded in #if GEN_GEN >= 6, and this code only
applies on Sandybridge.  So, use GEN_GEN == 6 to reduce the delta in
the next patch, when we add Gen4-5 support.

Reviewed-by: Rafael Antognolli <rafael.antognolli@intel.com>
src/mesa/drivers/dri/i965/genX_state_upload.c

index 6619d4d7ab1370e73e88002f16407255031d0fee..fccc66864a71254f0223165800a42a6f1e0355bf 100644 (file)
@@ -1731,6 +1731,7 @@ genX(upload_vs_state)(struct brw_context *brw)
    assert(vue_prog_data->dispatch_mode == DISPATCH_MODE_SIMD8 ||
           vue_prog_data->dispatch_mode == DISPATCH_MODE_4X2_DUAL_OBJECT);
 
+#if GEN_GEN == 6
    /* From the BSpec, 3D Pipeline > Geometry > Vertex Shader > State,
     * 3DSTATE_VS, Dword 5.0 "VS Function Enable":
     *
@@ -1742,8 +1743,6 @@ genX(upload_vs_state)(struct brw_context *brw)
     * We've already done such a flush at the start of state upload, so we
     * don't need to do another one here.
     */
-
-#if GEN_GEN < 7
    brw_batch_emit(brw, GENX(3DSTATE_CONSTANT_VS), cvs) {
       if (stage_state->push_const_size != 0) {
          cvs.Buffer0Valid = true;
@@ -1770,7 +1769,7 @@ genX(upload_vs_state)(struct brw_context *brw)
 #endif
    }
 
-#if GEN_GEN < 7
+#if GEN_GEN == 6
    /* Based on my reading of the simulator, the VS constants don't get
     * pulled into the VS FF unit until an appropriate pipeline flush
     * happens, and instead the 3DSTATE_CONSTANT_VS packet just adds
@@ -1797,12 +1796,12 @@ genX(upload_vs_state)(struct brw_context *brw)
 
 static const struct brw_tracked_state genX(vs_state) = {
    .dirty = {
-      .mesa  = (GEN_GEN < 7 ? (_NEW_PROGRAM_CONSTANTS | _NEW_TRANSFORM) : 0),
+      .mesa  = (GEN_GEN == 6 ? (_NEW_PROGRAM_CONSTANTS | _NEW_TRANSFORM) : 0),
       .brw   = BRW_NEW_BATCH |
                BRW_NEW_BLORP |
                BRW_NEW_CONTEXT |
                BRW_NEW_VS_PROG_DATA |
-               (GEN_GEN < 7 ? BRW_NEW_VERTEX_PROGRAM : 0),
+               (GEN_GEN == 6 ? BRW_NEW_VERTEX_PROGRAM : 0),
    },
    .emit = genX(upload_vs_state),
 };