i915g: handle varyings properly.
authorStéphane Marchesin <marcheu@chromium.org>
Sat, 4 Jun 2011 00:06:59 +0000 (17:06 -0700)
committerStéphane Marchesin <marcheu@chromium.org>
Mon, 6 Jun 2011 19:36:00 +0000 (12:36 -0700)
src/gallium/drivers/i915/i915_fpc_translate.c
src/gallium/drivers/i915/i915_state_derived.c

index b145b58be30aab28ee6329f561be824af56dbc6f..9f3c766cb81a0c77b42d1fed3fae3d1a499817ff 100644 (file)
@@ -192,8 +192,14 @@ src_vector(struct i915_fp_compile *p,
          src = swizzle(src, W, W, W, W);
          break;
       case TGSI_SEMANTIC_GENERIC:
-         /* usually a texcoord */
-         src = i915_emit_decl(p, REG_TYPE_T, T_TEX0 + sem_ind, D0_CHANNEL_ALL);
+         if (sem_ind < 8)
+            /* a texcoord */
+            src = i915_emit_decl(p, REG_TYPE_T, T_TEX0 + sem_ind, D0_CHANNEL_ALL);
+         else if ( (sem_ind >= 10) && (sem_ind < 18) )
+            /* a varying */
+            src = i915_emit_decl(p, REG_TYPE_T, T_TEX0 + sem_ind - 10, D0_CHANNEL_ALL);
+         else
+            debug_printf("%s: unhandled generic %d\n", __func__, sem_ind);
          break;
       default:
          i915_program_error(p, "Bad source->Index");
index 59ac2f7292a207f84b6ff8987e11df2d1802834f..47bea8bbd8be8c3131d8c496e176ce18e467cf83 100644 (file)
@@ -46,12 +46,12 @@ static void calculate_vertex_layout(struct i915_context *i915)
    const struct i915_fragment_shader *fs = i915->fs;
    const enum interp_mode colorInterp = i915->rasterizer->color_interp;
    struct vertex_info vinfo;
-   boolean texCoords[8], colors[2], fog, needW;
+   boolean texCoords[8], colors[2], fog, needW, have_varyings;
    uint i;
    int src;
 
    memset(texCoords, 0, sizeof(texCoords));
-   colors[0] = colors[1] = fog = needW = FALSE;
+   colors[0] = colors[1] = fog = needW = have_varyings = FALSE;
    memset(&vinfo, 0, sizeof(vinfo));
 
    /* Determine which fragment program inputs are needed.  Setup HW vertex
@@ -66,10 +66,19 @@ static void calculate_vertex_layout(struct i915_context *i915)
          colors[fs->info.input_semantic_index[i]] = TRUE;
          break;
       case TGSI_SEMANTIC_GENERIC:
-         /* usually a texcoord */
          {
-            const uint unit = fs->info.input_semantic_index[i];
+            /* texcoords/varyings */
+            /* XXX handle back/front face and point size */
+            uint unit = fs->info.input_semantic_index[i];
+
+            /* Route varyings as tex coords */
+            if ( (unit >= 10) && (unit < 18) ) {
+               have_varyings = TRUE;
+               unit -= 10;
+            }
+
             assert(unit < 8);
+
             texCoords[unit] = TRUE;
             needW = TRUE;
          }
@@ -82,7 +91,7 @@ static void calculate_vertex_layout(struct i915_context *i915)
       }
    }
 
-   
+
    /* pos */
    src = draw_find_shader_output(i915->draw, TGSI_SEMANTIC_POSITION, 0);
    if (needW) {
@@ -120,12 +129,15 @@ static void calculate_vertex_layout(struct i915_context *i915)
       vinfo.hwfmt[0] |= S4_VFMT_FOG_PARAM;
    }
 
-   /* texcoords */
+   /* texcoords/varyings */
    for (i = 0; i < 8; i++) {
       uint hwtc;
       if (texCoords[i]) {
          hwtc = TEXCOORDFMT_4D;
-         src = draw_find_shader_output(i915->draw, TGSI_SEMANTIC_GENERIC, i);
+         if (!have_varyings)
+            src = draw_find_shader_output(i915->draw, TGSI_SEMANTIC_GENERIC, i);
+         else
+            src = draw_find_shader_output(i915->draw, TGSI_SEMANTIC_GENERIC, i + 10);
          draw_emit_vertex_attr(&vinfo, EMIT_4F, INTERP_PERSPECTIVE, src);
       }
       else {