i965: Examine _BaseFormat when deciding to perform xRGB_alpha fixups
authorCarl Worth <cworth@cworth.org>
Mon, 21 Jan 2013 20:18:02 +0000 (12:18 -0800)
committerCarl Worth <cworth@cworth.org>
Thu, 24 Jan 2013 01:40:24 +0000 (17:40 -0800)
commitb961ba44ed01d4aa590609fc91ab1250ac7d9559
treebca58d3f515d6c6d1878b76aa5a294d296f96e79
parent06424376065a4a1e44920a29497936691d88598f
i965: Examine _BaseFormat when deciding to perform xRGB_alpha fixups

The renderbuffer's Format field may have an alpha channel even when the
underlying _BaseFormat does not. This can happen when mesa chooses to use
RGBA16 for an RGB16 format, for example.

So look at _BaseFormat when deciding whether to fixup the blend factors.

This test improves the results of at least the following piglit tests:

EXT_frambebuffer_object/fbo-blending-formats
         {GL_RGB10, GL_RGB12, GL_RGB16}
EXT_texture_snorm/fbo-blending-formats
{GL_RGB16_SNORM, GLRGB8_SNORM, GL_RGB_SNORM}

But none of these actually change from FAIL to PASS yet. The R, G, and B probe
values are fixed with this commit, but the tests still fail because the alpha
values are still wrong.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/gen6_cc.c