Translate mesa vertex/fragment programs to TGSI programs at same time to do proper linking.

Previously, programs were translated independently during validation.
The problem is the translation to TGSI format, which packs shader
input/outputs into continuous slots, depends on which vertex program is
being paired with which fragment shader.  Now, we look at the outputs
of the vertex program in conjunction with the inputs of the fragment shader
to be sure the attributes match up correctly.

The new 'linked_program_pair' class keeps track of the associations
between vertex and fragment shaders.  It's also the place where the TGSI
tokens are kept since they're no longer per-program state but per-linkage.

Still a few loose ends, like implementing some kind of hash/lookup table
for linked_program_pairs.

1 files changed, 4 insertions, 0 deletions

diff --git a/src/mesa/pipe/tgsi/mesa/mesa_to_tgsi.c b/src/mesa/pipe/tgsi/mesa/mesa_to_tgsi.c
index d0d97ab0f8..fa27fd3cd0 100644
--- a/src/mesa/pipe/tgsi/mesa/mesa_to_tgsi.c
+++ b/src/mesa/pipe/tgsi/mesa/mesa_to_tgsi.c
@@ -476,6 +476,8 @@ make_input_decl(
 {

    struct tgsi_full_declaration decl;

 

+   assert(semantic_name < TGSI_SEMANTIC_COUNT);

+

    decl = tgsi_default_full_declaration();

    decl.Declaration.File = TGSI_FILE_INPUT;

    decl.Declaration.Declare = TGSI_DECLARE_RANGE;

@@ -500,6 +502,8 @@ make_output_decl(
 {

    struct tgsi_full_declaration decl;

 

+   assert(semantic_name < TGSI_SEMANTIC_COUNT);

+

    decl = tgsi_default_full_declaration();

    decl.Declaration.File = TGSI_FILE_OUTPUT;

    decl.Declaration.Declare = TGSI_DECLARE_RANGE;