Mesa (master): i965: Drop the check for YUV constants in the param list.

Eric Anholt anholt at kemper.freedesktop.org
Fri Oct 8 17:25:39 UTC 2010


Module: Mesa
Branch: master
Commit: 0534e958c9d7fc854b64ad38964863e895e5a317
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0534e958c9d7fc854b64ad38964863e895e5a317

Author: Eric Anholt <eric at anholt.net>
Date:   Thu Oct  7 22:39:41 2010 -0700

i965: Drop the check for YUV constants in the param list.

_mesa_add_unnamed_constant() already does that.

---

 src/mesa/drivers/dri/i965/brw_wm_fp.c |   13 -------------
 1 files changed, 0 insertions(+), 13 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_wm_fp.c b/src/mesa/drivers/dri/i965/brw_wm_fp.c
index 19188a6..da3d348 100644
--- a/src/mesa/drivers/dri/i965/brw_wm_fp.c
+++ b/src/mesa/drivers/dri/i965/brw_wm_fp.c
@@ -555,19 +555,6 @@ static struct prog_src_register search_or_add_const4f( struct brw_wm_compile *c,
    values[2] = s2;
    values[3] = s3;
 
-   /* Have to search, otherwise multiple compilations will each grow
-    * the parameter list.
-    */
-   for (idx = 0; idx < paramList->NumParameters; idx++) {
-      if (paramList->Parameters[idx].Type == PROGRAM_CONSTANT &&
-	  memcmp(paramList->ParameterValues[idx], values, sizeof(values)) == 0)
-
-	 /* XXX: this mimics the mesa bug which puts all constants and
-	  * parameters into the "PROGRAM_STATE_VAR" category:
-	  */
-	 return src_reg(PROGRAM_STATE_VAR, idx);
-   }
-   
    idx = _mesa_add_unnamed_constant( paramList, values, 4, &swizzle );
    assert(swizzle == SWIZZLE_NOOP); /* Need to handle swizzle in reg setup */
    return src_reg(PROGRAM_STATE_VAR, idx);




More information about the mesa-commit mailing list