On Thu, 21 Jul 2011 22:23:42 -0700, Kenneth Graunke <kenn...@whitecape.org>
wrote:
> Commit 53c89c67f33639afef951e178f93f4e29acc5d53 ("i965: Avoid generating
> MOVs for assignments of expressions.") added the line "this->result =
> reg_undef" all over the code. Unfortunately, since Eric developed his
> patch before I landed Ivybridge support, he missed adding it to
> fs_visitor::emit_texture_gen7() after rebasing.
>
> Furthermore, since I developed TXD support before Eric's patch, I
> neglected to add it to the gradient handling when I rebased.
>
> Neglecting to set this causes the visitor to use this->result as storage
> rather than generating a new temporary. These missing statements
> resulted in the same register being used to store several different
> values.Reviewed-by: Eric Anholt <e...@anholt.net>
pgphdXZ5XJP3t.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
