i965/gen6: Fix point sprite texture coordinate overrides.
authorEric Anholt <eric@anholt.net>
Sun, 5 Jun 2011 18:50:00 +0000 (11:50 -0700)
committerEric Anholt <eric@anholt.net>
Thu, 9 Jun 2011 15:41:51 +0000 (08:41 -0700)
We were assuming that the input attribute n to the FS was
FRAG_ATTRIB_TEXn, which happened to be true often enough for our
testcases.

Acked-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/gen6_sf_state.c

index 45bd269914a85e2c04da8c969ac8977af2c57463..75a8509c87a5f640899937ffd43bc49d6b417cc5 100644 (file)
@@ -231,13 +231,6 @@ upload_sf_state(struct brw_context *brw)
         (1 << GEN6_SF_TRIFAN_PROVOKE_SHIFT);
    }
 
-   if (ctx->Point.PointSprite) {
-       for (i = 0; i < 8; i++) { 
-          if (ctx->Point.CoordReplace[i])
-              dw16 |= (1 << i);
-       }
-   }
-
    /* flat shading */
    if (ctx->Light.ShadeModel == GL_FLAT) {
        dw17 |= ((brw->fragment_program->Base.InputsRead & (FRAG_BIT_COL0 | FRAG_BIT_COL1)) >>
@@ -251,6 +244,13 @@ upload_sf_state(struct brw_context *brw)
       if (!(brw->fragment_program->Base.InputsRead & BITFIELD64_BIT(attr)))
         continue;
 
+      /* _NEW_POINT */
+      if (ctx->Point.PointSprite &&
+         (attr >= FRAG_ATTRIB_TEX0 && attr <= FRAG_ATTRIB_TEX7) &&
+         ctx->Point.CoordReplace[attr - FRAG_ATTRIB_TEX0]) {
+        dw16 |= (1 << input_index);
+      }
+
       /* The hardware can only do the overrides on 16 overrides at a
        * time, and the other up to 16 have to be lined up so that the
        * input index = the output index.  We'll need to do some