Kenneth Graunke <kenn...@whitecape.org> writes:

> While copying the values into the batch space, we advance the param
> pointer.  The debug code then tries to iterate over all the uploaded
> values, starting at param...which is now the end of the uploaded data,
> rather than the start.
>
> This patch saves a pointer to the start of push constant space before
> it gets altered and switches the debug code to use that.
>
> Tested by uncommenting the code and examining the output of
> glsl-vs-clamp-1.shader_test.  Previously all values appeared to be zero.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>

It would be nice if we just always uses_new_param_layout and stopped
incrementing the pointer, but I think you're looking at this code
because you're trying to do that.

Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpSgh6ftKj5f.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to