i965/gen8: Don't enable alpha test and alpha to coverage if draw bufer zero is intege...
authorAnuj Phogat <anuj.phogat@gmail.com>
Tue, 18 Oct 2016 22:52:19 +0000 (15:52 -0700)
committerAnuj Phogat <anuj.phogat@gmail.com>
Mon, 24 Oct 2016 18:07:39 +0000 (11:07 -0700)
We follow this rule at multiple places in i965 driver. This patch
doesn't fix any testcase.

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/gen8_blend_state.c

index 84cbf60c92f64da64acee946685574092a9a7861..c721da1a8f5c5886b58626799acb3819e10827e3 100644 (file)
@@ -218,13 +218,16 @@ gen8_upload_ps_blend(struct brw_context *brw)
    if (brw_color_buffer_write_enabled(brw))
       dw1 |= GEN8_PS_BLEND_HAS_WRITEABLE_RT;
 
-   /* _NEW_COLOR */
-   if (ctx->Color.AlphaEnabled)
-      dw1 |= GEN8_PS_BLEND_ALPHA_TEST_ENABLE;
+   if(!buffer0_is_integer) {
+      /* _NEW_COLOR */
+      if (ctx->Color.AlphaEnabled)
+         dw1 |= GEN8_PS_BLEND_ALPHA_TEST_ENABLE;
 
-   /* _NEW_MULTISAMPLE */
-   if (_mesa_is_multisample_enabled(ctx) && ctx->Multisample.SampleAlphaToCoverage)
-      dw1 |= GEN8_PS_BLEND_ALPHA_TO_COVERAGE_ENABLE;
+      /* _NEW_MULTISAMPLE */
+      if (_mesa_is_multisample_enabled(ctx) &&
+          ctx->Multisample.SampleAlphaToCoverage)
+         dw1 |= GEN8_PS_BLEND_ALPHA_TO_COVERAGE_ENABLE;
+   }
 
    /* Used for implementing the following bit of GL_EXT_texture_integer:
     * "Per-fragment operations that require floating-point color