On 02/22/2013 05:20 PM, Ian Romanick wrote:
On 02/22/2013 05:15 PM, Carl Worth wrote:
Ian Romanick <i...@freedesktop.org> writes:
Also, the correct way to say this is:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59445
Thanks. I missed that on this patch, but will get it right in the
future.
ctx->DrawBuffer->Visual doesn't know about your alpha bits when you
have
MRT some with alpha and some without. I prefer the first patch, which
would get my:
That makes sense. The probably means the other place that uses
ctx->DrawBuffer->Visual should get changed too.
I went to make that change, but there's no loop over draw buffers in
brw_cc.c. Presumably there's not support for MRT on that generation of
hardware?
We've supported MRT since GEN3... so there are two bugs in that other
code. :)
This also suggests that we need a MRT test case that can hit this path.
So I think we may be fine as-is here.
-Carl
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
