r300: bind vertex program to fragment program

diff --git a/src/mesa/drivers/dri/r300/r300_context.h b/src/mesa/drivers/dri/r300/r300_context.h
index 44211a45b31d1229903b4a20194831bff200e056..6f99e490500270f7cd3935f216c534a0d61f1ef6 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_context.h
+++ b/src/mesa/drivers/dri/r300/r300_context.h
@@ -409,9 +409,7 @@ struct r300_vertex_program {
        struct r300_vertex_program *next;
 
        struct r300_vertex_program_key {
-               GLuint InputsRead;
-               GLuint OutputsWritten;
-               GLuint OutputsAdded;
+               GLuint FpReads;
        } key;
        
        struct r300_vertex_shader_hw_code {

diff --git a/src/mesa/drivers/dri/r300/r300_draw.c b/src/mesa/drivers/dri/r300/r300_draw.c
index 4e8b62f1865c2bbed8da3c663cf12ade02e6bb9a..5420293b919308ced5608e24b6bfd58613aab23a 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_draw.c
+++ b/src/mesa/drivers/dri/r300/r300_draw.c
@@ -332,7 +332,7 @@ static void r300SetVertexFormat(GLcontext *ctx, const struct gl_client_array *ar
        {
                int i, tmp;
 
-               tmp = r300->selected_vp->key.InputsRead;
+               tmp = r300->selected_vp->Base->Base.InputsRead;
                i = 0;
                vbuf->num_attribs = 0;
                while (tmp) {
@@ -428,7 +428,7 @@ static GLboolean r300TryDrawPrims(GLcontext *ctx,
        if (r300->fallback)
                return GL_FALSE;
 
-       r300SetupVAP(ctx, r300->selected_vp->key.InputsRead, r300->selected_vp->key.OutputsWritten);
+       r300SetupVAP(ctx, r300->selected_vp->Base->Base.InputsRead, r300->selected_vp->Base->Base.OutputsWritten);
 
        r300UpdateShaderStates(r300);
 

diff --git a/src/mesa/drivers/dri/r300/r300_state.c b/src/mesa/drivers/dri/r300/r300_state.c
index 344f021acf6a601a2605f961f814e3444bafd2e0..0f3198e792d752497c1e679312d9ad660a503ce4 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_state.c
+++ b/src/mesa/drivers/dri/r300/r300_state.c
@@ -1456,7 +1456,7 @@ static void r300SetupRSUnit(GLcontext * ctx)
        hw_tcl_on = r300->options.hw_tcl_enabled;
 
        if (hw_tcl_on)
-               OutputsWritten.vp_outputs = r300->selected_vp->key.OutputsWritten;
+               OutputsWritten.vp_outputs = r300->selected_vp->Base->Base.OutputsWritten;
        else
                RENDERINPUTS_COPY(OutputsWritten.index_bitset, r300->render_inputs_bitset);
 
@@ -1573,7 +1573,7 @@ static void r500SetupRSUnit(GLcontext * ctx)
        hw_tcl_on = r300->options.hw_tcl_enabled;
 
        if (hw_tcl_on)
-               OutputsWritten.vp_outputs = r300->selected_vp->key.OutputsWritten;
+               OutputsWritten.vp_outputs = r300->selected_vp->Base->Base.OutputsWritten;
        else
                RENDERINPUTS_COPY(OutputsWritten.index_bitset, r300->render_inputs_bitset);
 

diff --git a/src/mesa/drivers/dri/r300/r300_vertprog.c b/src/mesa/drivers/dri/r300/r300_vertprog.c
index 326183bda42a2572cea9a1f175dcf391148549de..0b048303508c9b68cbd3743c64965b4abfc0ca39 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_vertprog.c
+++ b/src/mesa/drivers/dri/r300/r300_vertprog.c
@@ -961,10 +961,14 @@ static void t_inputs_outputs(struct r300_vertex_program *vp)
 {
        int i;
        int cur_reg;
+       GLuint OutputsWritten, InputsRead;
+
+       OutputsWritten = vp->Base->Base.OutputsWritten;
+       InputsRead = vp->Base->Base.InputsRead;
 
        cur_reg = -1;
        for (i = 0; i < VERT_ATTRIB_MAX; i++) {
-               if (vp->key.InputsRead & (1 << i))
+               if (InputsRead & (1 << i))
                        vp->inputs[i] = ++cur_reg;
                else
                        vp->inputs[i] = -1;
@@ -974,13 +978,13 @@ static void t_inputs_outputs(struct r300_vertex_program *vp)
        for (i = 0; i < VERT_RESULT_MAX; i++)
                vp->outputs[i] = -1;
 
-       assert(vp->key.OutputsWritten & (1 << VERT_RESULT_HPOS));
+       assert(OutputsWritten & (1 << VERT_RESULT_HPOS));
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_HPOS)) {
+       if (OutputsWritten & (1 << VERT_RESULT_HPOS)) {
                vp->outputs[VERT_RESULT_HPOS] = cur_reg++;
        }
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_PSIZ)) {
+       if (OutputsWritten & (1 << VERT_RESULT_PSIZ)) {
                vp->outputs[VERT_RESULT_PSIZ] = cur_reg++;
        }
 
@@ -990,39 +994,39 @@ static void t_inputs_outputs(struct r300_vertex_program *vp)
         * pretend it does by skipping output index reg so the colors
         * get written into appropriate output vectors.
         */
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_COL0)) {
+       if (OutputsWritten & (1 << VERT_RESULT_COL0)) {
                vp->outputs[VERT_RESULT_COL0] = cur_reg++;
-       } else if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC0) ||
-               vp->key.OutputsWritten & (1 << VERT_RESULT_BFC1)) {
+       } else if (OutputsWritten & (1 << VERT_RESULT_BFC0) ||
+               OutputsWritten & (1 << VERT_RESULT_BFC1)) {
                cur_reg++;
        }
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_COL1)) {
+       if (OutputsWritten & (1 << VERT_RESULT_COL1)) {
                vp->outputs[VERT_RESULT_COL1] = cur_reg++;
-       } else if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC0) ||
-               vp->key.OutputsWritten & (1 << VERT_RESULT_BFC1)) {
+       } else if (OutputsWritten & (1 << VERT_RESULT_BFC0) ||
+               OutputsWritten & (1 << VERT_RESULT_BFC1)) {
                cur_reg++;
        }
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC0)) {
+       if (OutputsWritten & (1 << VERT_RESULT_BFC0)) {
                vp->outputs[VERT_RESULT_BFC0] = cur_reg++;
-       } else if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC1)) {
+       } else if (OutputsWritten & (1 << VERT_RESULT_BFC1)) {
                cur_reg++;
        }
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC1)) {
+       if (OutputsWritten & (1 << VERT_RESULT_BFC1)) {
                vp->outputs[VERT_RESULT_BFC1] = cur_reg++;
-       } else if (vp->key.OutputsWritten & (1 << VERT_RESULT_BFC0)) {
+       } else if (OutputsWritten & (1 << VERT_RESULT_BFC0)) {
                cur_reg++;
        }
 
        for (i = VERT_RESULT_TEX0; i <= VERT_RESULT_TEX7; i++) {
-               if (vp->key.OutputsWritten & (1 << i)) {
+               if (OutputsWritten & (1 << i)) {
                        vp->outputs[i] = cur_reg++;
                }
        }
 
-       if (vp->key.OutputsWritten & (1 << VERT_RESULT_FOGC)) {
+       if (OutputsWritten & (1 << VERT_RESULT_FOGC)) {
                vp->outputs[VERT_RESULT_FOGC] = cur_reg++;
        }
 }
@@ -1255,6 +1259,8 @@ static void insert_wpos(struct r300_vertex_program *vp, struct gl_program *prog,
        ++vpi;
 
        vpi->Opcode = OPCODE_END;
+
+       prog->OutputsWritten |= 1 << (VERT_RESULT_TEX0 + vp->wpos_idx);
 }
 
 static void pos_as_texcoord(struct r300_vertex_program *vp,
@@ -1448,6 +1454,63 @@ static void translateInsts(struct gl_program *prog)
        }
 }
 
+#define ADD_OUTPUT(fp_attr, vp_result) \
+       do { \
+               if ((FpReads & (1 << (fp_attr))) && !(prog->OutputsWritten & (1 << (vp_result)))) { \
+                       OutputsAdded |= 1 << (vp_result); \
+                       count++; \
+               } \
+       } while (0)
+
+static void addArtificialOutputs(GLcontext *ctx, struct gl_program *prog)
+{
+       r300ContextPtr r300 = R300_CONTEXT(ctx);
+       GLuint OutputsAdded, FpReads;
+       int i, count;
+
+       OutputsAdded = count = 0;
+       FpReads = r300->selected_fp->Base->InputsRead;
+
+       ADD_OUTPUT(FRAG_ATTRIB_COL0, VERT_RESULT_COL0);
+       ADD_OUTPUT(FRAG_ATTRIB_COL1, VERT_RESULT_COL1);
+
+       for (i = 0; i < 7; ++i) {
+               ADD_OUTPUT(FRAG_ATTRIB_TEX0 + i, VERT_RESULT_TEX0 + i);
+       }
+
+       /* Some outputs may be artificially added, to match the inputs of the fragment program.
+        * Issue 16 of vertex program spec says that all vertex attributes that are unwritten by
+        * vertex program are undefined, so just use MOV [vertex_result], CONST[0]
+        */
+       if (count > 0) {
+               struct prog_instruction *inst;
+
+               _mesa_insert_instructions(prog, prog->NumInstructions - 1, count);
+               inst = &prog->Instructions[prog->NumInstructions - 1 - count];
+
+               for (i = 0; i < VERT_RESULT_MAX; ++i) {
+                       if (OutputsAdded & (1 << i)) {
+                               inst->Opcode = OPCODE_MOV;
+
+                               inst->DstReg.File = PROGRAM_OUTPUT;
+                               inst->DstReg.Index = i;
+                               inst->DstReg.WriteMask = WRITEMASK_XYZW;
+                               inst->DstReg.CondMask = COND_TR;
+
+                               inst->SrcReg[0].File = PROGRAM_CONSTANT;
+                               inst->SrcReg[0].Index = 0;
+                               inst->SrcReg[0].Swizzle = SWIZZLE_XYZW;
+
+                               ++inst;
+                       }
+               }
+
+               prog->OutputsWritten |= OutputsAdded;
+       }
+}
+
+#undef ADD_OUTPUT
+
 static struct r300_vertex_program *build_program(GLcontext *ctx,
                                                 struct r300_vertex_program_key *wanted_key,
                                                 const struct gl_vertex_program *mesa_vp,
@@ -1477,42 +1540,7 @@ static struct r300_vertex_program *build_program(GLcontext *ctx,
                pos_as_texcoord(vp, prog);
        }
 
-       /* Some outputs may be artificially added, to match the inputs of the fragment program.
-        * Issue 16 of vertex program spec says that all vertex attributes that are unwritten by
-        * vertex program are undefined, so just use MOV [vertex_result], CONST[0]
-        */
-       {
-               int i, count = 0;
-               for (i = 0; i < VERT_RESULT_MAX; ++i) {
-                       if (vp->key.OutputsAdded & (1 << i)) {
-                               ++count;
-                       }
-               }
-
-               if (count > 0) {
-                       struct prog_instruction *inst;
-
-                       _mesa_insert_instructions(prog, prog->NumInstructions - 1, count);
-                       inst = &prog->Instructions[prog->NumInstructions - 1 - count];
-
-                       for (i = 0; i < VERT_RESULT_MAX; ++i) {
-                               if (vp->key.OutputsAdded & (1 << i)) {
-                                       inst->Opcode = OPCODE_MOV;
-
-                                       inst->DstReg.File = PROGRAM_OUTPUT;
-                                       inst->DstReg.Index = i;
-                                       inst->DstReg.WriteMask = WRITEMASK_XYZW;
-                                       inst->DstReg.CondMask = COND_TR;
-
-                                       inst->SrcReg[0].File = PROGRAM_CONSTANT;
-                                       inst->SrcReg[0].Index = 0;
-                                       inst->SrcReg[0].Swizzle = SWIZZLE_XYZW;
-
-                                       ++inst;
-                               }
-                       }
-               }
-       }
+       addArtificialOutputs(ctx, prog);
 
        translateInsts(prog);
 
@@ -1528,76 +1556,39 @@ static struct r300_vertex_program *build_program(GLcontext *ctx,
        return vp;
 }
 
-static void add_outputs(struct r300_vertex_program_key *key, GLint vert)
-{
-       if (key->OutputsWritten & (1 << vert))
-               return;
-
-       key->OutputsWritten |= 1 << vert;
-       key->OutputsAdded |= 1 << vert;
-}
-
 struct r300_vertex_program * r300SelectVertexShader(GLcontext *ctx)
 {
        r300ContextPtr r300 = R300_CONTEXT(ctx);
-       GLuint InputsRead;
        struct r300_vertex_program_key wanted_key = { 0 };
-       GLint i;
        struct r300_vertex_program_cont *vpc;
        struct r300_vertex_program *vp;
        GLint wpos_idx;
 
        vpc = (struct r300_vertex_program_cont *)ctx->VertexProgram._Current;
-       wanted_key.InputsRead = vpc->mesa_program.Base.InputsRead;
-       wanted_key.OutputsWritten = vpc->mesa_program.Base.OutputsWritten;
-       InputsRead = ctx->FragmentProgram._Current->Base.InputsRead;
-
-       wpos_idx = -1;
-       if (InputsRead & FRAG_BIT_WPOS) {
-               for (i = 0; i < ctx->Const.MaxTextureUnits; i++)
-                       if (!(InputsRead & (FRAG_BIT_TEX0 << i)))
-                               break;
+       wanted_key.FpReads = r300->selected_fp->Base->InputsRead;
 
-               if (i == ctx->Const.MaxTextureUnits) {
-                       fprintf(stderr, "\tno free texcoord found\n");
-                       _mesa_exit(-1);
+       for (vp = vpc->progs; vp; vp = vp->next)
+               if (_mesa_memcmp(&vp->key, &wanted_key, sizeof(wanted_key))
+                   == 0) {
+                       return r300->selected_vp = vp;
                }
 
-               wanted_key.OutputsWritten |= 1 << (VERT_RESULT_TEX0 + i);
-               wpos_idx = i;
-       }
-
-       if (vpc->mesa_program.IsPositionInvariant) {
-               wanted_key.InputsRead |= (1 << VERT_ATTRIB_POS);
-               wanted_key.OutputsWritten |= (1 << VERT_RESULT_HPOS);
-       } else {
-               add_outputs(&wanted_key, VERT_RESULT_HPOS);
-       }
-
-       if (InputsRead & FRAG_BIT_COL0) {
-               add_outputs(&wanted_key, VERT_RESULT_COL0);
-       }
+       wpos_idx = -1;
+       if (wanted_key.FpReads & FRAG_BIT_WPOS) {
+               GLint i;
 
-       if (InputsRead & FRAG_BIT_COL1) {
-               add_outputs(&wanted_key, VERT_RESULT_COL1);
-       }
+               for (i = 0; i < ctx->Const.MaxTextureUnits; i++)
+                       if (!(wanted_key.FpReads & (FRAG_BIT_TEX(i))))
+                               break;
 
-       if (InputsRead & FRAG_BIT_FOGC) {
-               add_outputs(&wanted_key, VERT_RESULT_FOGC);
-       }
+                       if (i == ctx->Const.MaxTextureUnits) {
+                               fprintf(stderr, "\tno free texcoord found\n");
+                               _mesa_exit(-1);
+                       }
 
-       for (i = 0; i < ctx->Const.MaxTextureUnits; i++) {
-               if (InputsRead & (FRAG_BIT_TEX0 << i)) {
-                       add_outputs(&wanted_key, VERT_RESULT_TEX0 + i);
-               }
+                       wpos_idx = i;
        }
 
-       for (vp = vpc->progs; vp; vp = vp->next)
-               if (_mesa_memcmp(&vp->key, &wanted_key, sizeof(wanted_key))
-                   == 0) {
-                       return r300->selected_vp = vp;
-               }
-
        vp = build_program(ctx, &wanted_key, &vpc->mesa_program, wpos_idx);
        vp->next = vpc->progs;
        vpc->progs = vp;
@@ -1663,8 +1654,8 @@ void r300SetupVertexProgram(r300ContextPtr rmesa)
        r300EmitVertexProgram(rmesa, R300_PVS_CODE_START, &(prog->hw_code));
        inst_count = (prog->hw_code.length / 4) - 1;
 
-       r300VapCntl(rmesa, _mesa_bitcount(prog->key.InputsRead),
-                                _mesa_bitcount(prog->key.OutputsWritten), prog->num_temporaries);
+       r300VapCntl(rmesa, _mesa_bitcount(prog->Base->Base.InputsRead),
+                                _mesa_bitcount(prog->Base->Base.OutputsWritten), prog->num_temporaries);
 
        R300_STATECHANGE(rmesa, pvs);
        rmesa->hw.pvs.cmd[R300_PVS_CNTL_1] = (0 << R300_PVS_FIRST_INST_SHIFT) | (inst_count << R300_PVS_XYZW_VALID_INST_SHIFT) |