st/mesa: Remove support for GL_EXT_blend_logic_op
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 9 Sep 2011 21:07:54 +0000 (14:07 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Mon, 19 Sep 2011 19:02:20 +0000 (12:02 -0700)
It was broken, and it isn't really useful anyway.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Marek Olšák <maraeo@gmail.com>
Cc: Brian Paul <brianp@vmware.com>
src/mesa/state_tracker/st_atom_blend.c
src/mesa/state_tracker/st_extensions.c

index 4c9a2b95ac304f834f5bfb7f53479ea954d1053a..1ffc2f1b73ff506e0456d380a96936328c130267 100644 (file)
@@ -200,15 +200,7 @@ update_blend( struct st_context *st )
       num_state = ctx->Const.MaxDrawBuffers;
       blend->independent_blend_enable = 1;
    }
-   /* Note it is impossible to correctly deal with EXT_blend_logic_op and
-      EXT_draw_buffers2/EXT_blend_equation_separate at the same time.
-      These combinations would require support for per-rt logicop enables
-      and separate alpha/rgb logicop/blend support respectively. Neither
-      possible in gallium nor most hardware. Assume these combinations
-      don't happen. */
-   if (ctx->Color.ColorLogicOpEnabled ||
-       (ctx->Color.BlendEnabled &&
-        ctx->Color.Blend[0].EquationRGB == GL_LOGIC_OP)) {
+   if (ctx->Color.ColorLogicOpEnabled) {
       /* logicop enabled */
       blend->logicop_enable = 1;
       blend->logicop_func = translate_logicop(ctx->Color.LogicOp);
index 722db8d774f004de62c49f96f383c10b70029114..108d7aef15bea3b1e4969151ea563228155bae42 100644 (file)
@@ -267,7 +267,6 @@ void st_init_extensions(struct st_context *st)
 
    ctx->Extensions.EXT_blend_color = GL_TRUE;
    ctx->Extensions.EXT_blend_func_separate = GL_TRUE;
-   ctx->Extensions.EXT_blend_logic_op = GL_TRUE;
    ctx->Extensions.EXT_blend_minmax = GL_TRUE;
    ctx->Extensions.EXT_blend_subtract = GL_TRUE;
    ctx->Extensions.EXT_framebuffer_blit = GL_TRUE;