i965: Cleanly fail programs with unsupported array access.
authorEric Anholt <eric@anholt.net>
Fri, 23 Jul 2010 17:01:45 +0000 (10:01 -0700)
committerEric Anholt <eric@anholt.net>
Fri, 23 Jul 2010 17:54:08 +0000 (10:54 -0700)
This should be more useful for developers and for bug triaging than
just generating wrong code.

src/mesa/drivers/dri/i965/brw_program.c

index aeed24d4e14792cf161fc1dee2d84f048a62e09c..4b08d2599bc0b1c49546856366f437a6dc80cb15 100644 (file)
@@ -174,9 +174,36 @@ static GLboolean brwProgramStringNotify( GLcontext *ctx,
         shader_error(ctx, prog,
                      "i965 driver doesn't yet support uninlined function "
                      "calls.  Move to using a single return statement at "
-                     "the end of the function to work around it.");
+                     "the end of the function to work around it.\n");
         return GL_FALSE;
       }
+      if (prog->Instructions[i].DstReg.RelAddr &&
+         prog->Instructions[i].DstReg.File == PROGRAM_INPUT) {
+        shader_error(ctx, prog,
+                     "Variable indexing of shader inputs unsupported\n");
+        return GL_FALSE;
+      }
+      if (prog->Instructions[i].DstReg.RelAddr &&
+         prog->Instructions[i].DstReg.File == PROGRAM_OUTPUT) {
+        shader_error(ctx, prog,
+                     "Variable indexing of shader outputs unsupported\n");
+        return GL_FALSE;
+      }
+      if (target == GL_FRAGMENT_PROGRAM_ARB) {
+        if ((prog->Instructions[i].DstReg.RelAddr &&
+             prog->Instructions[i].DstReg.File == PROGRAM_TEMPORARY) ||
+            (prog->Instructions[i].SrcReg[0].RelAddr &&
+             prog->Instructions[i].SrcReg[0].File == PROGRAM_TEMPORARY) ||
+            (prog->Instructions[i].SrcReg[1].RelAddr &&
+             prog->Instructions[i].SrcReg[1].File == PROGRAM_TEMPORARY) ||
+            (prog->Instructions[i].SrcReg[2].RelAddr &&
+             prog->Instructions[i].SrcReg[2].File == PROGRAM_TEMPORARY)) {
+           shader_error(ctx, prog,
+                        "Variable indexing of variable arrays in the FS "
+                        "unsupported\n");
+           return GL_FALSE;
+        }
+      }
    }
 
    return GL_TRUE;