From: Eric Anholt <eric@anholt.net>
Date: Fri, 8 Oct 2010 05:39:41 +0000 (-0700)
Subject: i965: Drop the check for YUV constants in the param list.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0534e958c9d7fc854b64ad38964863e895e5a317;p=mesa.git

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

_mesa_add_unnamed_constant() already does that.
---

diff --git a/src/mesa/drivers/dri/i965/brw_wm_fp.c b/src/mesa/drivers/dri/i965/brw_wm_fp.c
index 19188a67972..da3d3486cea 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);