Looking at the spec, that seems like that should also check for sRGB and also disable in that case (“GetFormatInfo(compileState.format).isSRGB”).
> On Nov 7, 2016, at 6:18 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote: > > Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu> > --- > src/gallium/drivers/swr/swr_state.cpp | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/src/gallium/drivers/swr/swr_state.cpp > b/src/gallium/drivers/swr/swr_state.cpp > index d8a8ee1..acb0452 100644 > --- a/src/gallium/drivers/swr/swr_state.cpp > +++ b/src/gallium/drivers/swr/swr_state.cpp > @@ -1305,6 +1305,11 @@ swr_update_derived(struct pipe_context *pipe, > &ctx->blend->compileState[target], > sizeof(compileState.blendState)); > > + if (compileState.blendState.logicOpEnable && > + GetFormatInfo(compileState.format).type[0] == > SWR_TYPE_FLOAT) { > + compileState.blendState.logicOpEnable = false; > + } > + > if (compileState.blendState.blendEnable == false && > compileState.blendState.logicOpEnable == false) { > SwrSetBlendFunc(ctx->swrContext, target, NULL); > -- > 2.7.3 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev