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