mesa: change constant search behaviour in _mesa_add_unnamed_constant()
authorBrian Paul <brianp@vmware.com>
Tue, 5 Jan 2010 16:57:41 +0000 (09:57 -0700)
committerBrian Paul <brianp@vmware.com>
Tue, 5 Jan 2010 18:44:19 +0000 (11:44 -0700)
Only search for an existing, matching constant if swizzleOut is non-null.
We need to be able to disable constant consolidation when building an
array of constants which will be indexed indirectly.

src/mesa/shader/prog_parameter.c

index f22492e029e3a5e1c23a53f619bdf3b82a1e9308..5822510701bd88dac446cdccb4062f38cc3074e2 100644 (file)
@@ -230,9 +230,8 @@ _mesa_add_named_constant(struct gl_program_parameter_list *paramList,
  * Add a new unnamed constant to the parameter list.  This will be used
  * when a fragment/vertex program contains something like this:
  *    MOV r, { 0, 1, 2, 3 };
- * We'll search the parameter list for an existing instance of the
- * constant.  If swizzleOut is non-null, we'll try swizzling when
- * looking for a match.
+ * If swizzleOut is non-null we'll search the parameter list for an
+ * existing instance of the constant which matches with a swizzle.
  *
  * \param paramList  the parameter list
  * \param values  four float values
@@ -248,7 +247,8 @@ _mesa_add_unnamed_constant(struct gl_program_parameter_list *paramList,
    ASSERT(size >= 1);
    ASSERT(size <= 4);
 
-   if (_mesa_lookup_parameter_constant(paramList, values,
+   if (swizzleOut &&
+       _mesa_lookup_parameter_constant(paramList, values,
                                        size, &pos, swizzleOut)) {
       return pos;
    }