Revert "r600g: precalculate semantic indices for SPI setup"
authorAlex Deucher <alexander.deucher@amd.com>
Fri, 4 Nov 2011 22:33:31 +0000 (18:33 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 4 Nov 2011 22:33:31 +0000 (18:33 -0400)
This reverts commit c15f8569fddac5f8aee77863922fd5bb992cfe8a.

This breaks r6xx.

src/gallium/drivers/r600/evergreen_state.c
src/gallium/drivers/r600/r600_shader.c
src/gallium/drivers/r600/r600_shader.h
src/gallium/drivers/r600/r600_state.c
src/gallium/drivers/r600/r600_state_common.c

index 904267da1bd714bb490e35fe83041fd03e803f64..fd2e5da8664a73571fbc34cc092ee77f19ded7fc 100644 (file)
@@ -2375,20 +2375,20 @@ void evergreen_pipe_shader_vs(struct pipe_context *ctx, struct r600_pipe_shader
        struct r600_pipe_context *rctx = (struct r600_pipe_context *)ctx;
        struct r600_pipe_state *rstate = &shader->rstate;
        struct r600_shader *rshader = &shader->shader;
-       unsigned spi_vs_out_id[10] = {};
-       unsigned i, tmp, nparams = 0;
+       unsigned spi_vs_out_id[10];
+       unsigned i, tmp, nparams;
 
        /* clear previous register */
        rstate->nregs = 0;
 
-       for (i = 0; i < rshader->noutput; i++) {
-               if (rshader->output[i].spi_sid) {
-                       tmp = rshader->output[i].spi_sid << ((nparams & 3) * 8);
-                       spi_vs_out_id[nparams / 4] |= tmp;
-                       nparams++;
-               }
+       /* so far never got proper semantic id from tgsi */
+       for (i = 0; i < 10; i++) {
+               spi_vs_out_id[i] = 0;
+       }
+       for (i = 0; i < 32; i++) {
+               tmp = i << ((i & 3) * 8);
+               spi_vs_out_id[i / 4] |= tmp;
        }
-
        for (i = 0; i < 10; i++) {
                r600_pipe_state_add_reg(rstate,
                                        R_02861C_SPI_VS_OUT_ID_0 + i * 4,
@@ -2399,6 +2399,7 @@ void evergreen_pipe_shader_vs(struct pipe_context *ctx, struct r600_pipe_shader
         * VS is required to export at least one param and r600_shader_from_tgsi()
         * takes care of adding a dummy export.
         */
+       nparams = rshader->noutput - rshader->npos;
        if (nparams < 1)
                nparams = 1;
 
index 448e03ae39e7685e3180124973f175a16b96580b..87164ce48df491733191a1f1e6691bb6ad60f7f4 100644 (file)
@@ -57,6 +57,24 @@ issued in the w slot as well.
 The compiler must issue the source argument to slots z, y, and x
 */
 
+
+int r600_find_vs_semantic_index(struct r600_shader *vs,
+                               struct r600_shader *ps, int id)
+{
+       struct r600_shader_io *input = &ps->input[id];
+       int index = 0;
+
+       for (int i = 0; i < vs->noutput; i++) {
+               if (input->name == vs->output[i].name &&
+                               input->sid == vs->output[i].sid)
+                       return index;
+               else if (vs->output[i].name != TGSI_SEMANTIC_POSITION &&
+                                vs->output[i].name != TGSI_SEMANTIC_PSIZE)
+                       index++;
+       }
+       return 0;
+}
+
 static int r600_pipe_shader(struct pipe_context *ctx, struct r600_pipe_shader *shader)
 {
        struct r600_pipe_context *rctx = (struct r600_pipe_context *)ctx;
@@ -343,44 +361,6 @@ static int evergreen_interp_flat(struct r600_shader_ctx *ctx, int input)
  * DB_SOURCE_FORMAT - export control restrictions
  *
  */
-
-
-/* Map name/sid pair from tgsi to the 8-bit semantic index for SPI setup */
-static int r600_spi_sid(struct r600_shader_io * io)
-{
-       int index, name = io->name;
-
-       /* These params are handled differently, they don't need
-        * semantic indices, so we'll use 0 for them.
-        */
-       if (name == TGSI_SEMANTIC_POSITION ||
-               name == TGSI_SEMANTIC_PSIZE ||
-               name == TGSI_SEMANTIC_FACE)
-               index = 0;
-       else {
-               if (name == TGSI_SEMANTIC_GENERIC) {
-                       /* For generic params simply use sid from tgsi */
-                       index = io->sid;
-               } else {
-
-                       /* FIXME: two-side rendering is broken in r600g, this will
-                        * keep old functionality */
-                       if (name == TGSI_SEMANTIC_BCOLOR)
-                               name = TGSI_SEMANTIC_COLOR;
-
-                       /* For non-generic params - pack name and sid into 8 bits */
-                       index = 0x80 | (name<<3) | (io->sid);
-               }
-
-               /* Make sure that all really used indices have nonzero value, so
-                * we can just compare it to 0 later instead of comparing the name
-                * with different values to detect special cases. */
-               index++;
-       }
-
-       return index;
-};
-
 static int tgsi_declaration(struct r600_shader_ctx *ctx)
 {
        struct tgsi_full_declaration *d = &ctx->parse.FullToken.FullDeclaration;
@@ -392,13 +372,13 @@ static int tgsi_declaration(struct r600_shader_ctx *ctx)
                i = ctx->shader->ninput++;
                ctx->shader->input[i].name = d->Semantic.Name;
                ctx->shader->input[i].sid = d->Semantic.Index;
-               ctx->shader->input[i].spi_sid = r600_spi_sid(&ctx->shader->input[i]);
                ctx->shader->input[i].interpolate = d->Declaration.Interpolate;
                ctx->shader->input[i].centroid = d->Declaration.Centroid;
                ctx->shader->input[i].gpr = ctx->file_offset[TGSI_FILE_INPUT] + d->Range.First;
                if (ctx->type == TGSI_PROCESSOR_FRAGMENT && ctx->bc->chip_class >= EVERGREEN) {
                        /* turn input into interpolate on EG */
-                       if (ctx->shader->input[i].spi_sid) {
+                       if (ctx->shader->input[i].name != TGSI_SEMANTIC_POSITION &&
+                           ctx->shader->input[i].name != TGSI_SEMANTIC_FACE) {
                                ctx->shader->input[i].lds_pos = ctx->shader->nlds++;
                                if (ctx->shader->input[i].interpolate > 0) {
                                        evergreen_interp_alu(ctx, i);
@@ -412,9 +392,14 @@ static int tgsi_declaration(struct r600_shader_ctx *ctx)
                i = ctx->shader->noutput++;
                ctx->shader->output[i].name = d->Semantic.Name;
                ctx->shader->output[i].sid = d->Semantic.Index;
-               ctx->shader->output[i].spi_sid = r600_spi_sid(&ctx->shader->output[i]);
                ctx->shader->output[i].gpr = ctx->file_offset[TGSI_FILE_OUTPUT] + d->Range.First;
                ctx->shader->output[i].interpolate = d->Declaration.Interpolate;
+               if (ctx->type == TGSI_PROCESSOR_VERTEX) {
+                       /* these don't count as vertex param exports */
+                       if ((ctx->shader->output[i].name == TGSI_SEMANTIC_POSITION) ||
+                           (ctx->shader->output[i].name == TGSI_SEMANTIC_PSIZE))
+                               ctx->shader->npos++;
+               }
                break;
        case TGSI_FILE_CONSTANT:
        case TGSI_FILE_TEMPORARY:
index 9990ba646af6958603faab7cfe373ff159a784c2..ada369ade68837511e5d31d14255cecb46613ec9 100644 (file)
@@ -30,7 +30,6 @@ struct r600_shader_io {
        unsigned                gpr;
        unsigned                done;
        int                     sid;
-       int                     spi_sid;
        unsigned                interpolate;
        boolean                 centroid;
        unsigned                lds_pos; /* for evergreen */
@@ -41,6 +40,7 @@ struct r600_shader {
        struct r600_bytecode            bc;
        unsigned                ninput;
        unsigned                noutput;
+       unsigned                npos;
        unsigned                nlds;
        struct r600_shader_io   input[32];
        struct r600_shader_io   output[32];
index ce65da668df0d2aeb5909c5991e0c3d8eb822acc..bf3da791c8156ddc91b438c152eb73a37be9b36e 100644 (file)
@@ -2142,19 +2142,22 @@ void r600_pipe_shader_vs(struct pipe_context *ctx, struct r600_pipe_shader *shad
        struct r600_pipe_state *rstate = &shader->rstate;
        struct r600_shader *rshader = &shader->shader;
        unsigned spi_vs_out_id[10];
-       unsigned i, tmp, nparams = 0;
+       unsigned i, tmp, nparams;
 
        /* clear previous register */
        rstate->nregs = 0;
 
-       for (i = 0; i < rshader->noutput; i++) {
-               if (rshader->output[i].spi_sid) {
-                       tmp = rshader->output[i].spi_sid << ((nparams & 3) * 8);
-                       spi_vs_out_id[nparams / 4] |= tmp;
-                       nparams++;
-               }
+       /* so far never got proper semantic id from tgsi */
+       /* FIXME better to move this in config things so they get emited
+        * only one time per cs
+        */
+       for (i = 0; i < 10; i++) {
+               spi_vs_out_id[i] = 0;
+       }
+       for (i = 0; i < 32; i++) {
+               tmp = i << ((i & 3) * 8);
+               spi_vs_out_id[i / 4] |= tmp;
        }
-
        for (i = 0; i < 10; i++) {
                r600_pipe_state_add_reg(rstate,
                                        R_028614_SPI_VS_OUT_ID_0 + i * 4,
@@ -2165,6 +2168,7 @@ void r600_pipe_shader_vs(struct pipe_context *ctx, struct r600_pipe_shader *shad
         * VS is required to export at least one param and r600_shader_from_tgsi()
         * takes care of adding a dummy export.
         */
+       nparams = rshader->noutput - rshader->npos;
        if (nparams < 1)
                nparams = 1;
 
index 3d0345addce1f6d2e69b3adc7bf32b19ee2bc2f5..b24bb549371d0b920e1b0aec3f6e557457b494f3 100644 (file)
@@ -366,11 +366,14 @@ static void r600_spi_update(struct r600_pipe_context *rctx)
 
        rstate->nregs = 0;
        for (i = 0; i < rshader->ninput; i++) {
-
-               sid = rshader->input[i].spi_sid;
-
-               if (!sid && (rctx->chip_class >= EVERGREEN))
-                       continue;
+               if (rshader->input[i].name == TGSI_SEMANTIC_POSITION ||
+                   rshader->input[i].name == TGSI_SEMANTIC_FACE)
+                       if (rctx->chip_class >= EVERGREEN)
+                               continue;
+                       else
+                               sid=0;
+               else
+                       sid=r600_find_vs_semantic_index(&rctx->vs_shader->shader, rshader, i);
 
                tmp = S_028644_SEMANTIC(sid);