[Mesa-dev] [PATCH 3/3] i965/gen8: Don't enable alpha test and alpha to coverage if draw bufer zero is integer type

Kenneth Graunke kenneth at whitecape.org
Sun Oct 23 08:42:37 UTC 2016


On Friday, October 21, 2016 2:18:00 PM PDT Anuj Phogat wrote:
> We follow this rule at multiple places in i965 driver. This patch
> doesn't fix any testcase.
> 
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  src/mesa/drivers/dri/i965/gen8_blend_state.c | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/gen8_blend_state.c b/src/mesa/drivers/dri/i965/gen8_blend_state.c
> index 84cbf60..c721da1 100644
> --- a/src/mesa/drivers/dri/i965/gen8_blend_state.c
> +++ b/src/mesa/drivers/dri/i965/gen8_blend_state.c
> @@ -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) {

      ^^^ space before paren

Nice cleanup.

Series is:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161023/0b64f955/attachment.sig>


More information about the mesa-dev mailing list