i965: Fix copy propagation type changes.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 2 Sep 2015 23:39:27 +0000 (16:39 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 4 Sep 2015 04:12:54 +0000 (21:12 -0700)
commit2ace64fd598816fd1be9877962734242fc27b87b
tree95bc00d6f81cadf845e701253323fee6970ce52c
parent5fa5a012b163bad80d01a65e2947510f15e1be6b
i965: Fix copy propagation type changes.

commit 472ef9a02f2e5c5d0caa2809cb736a0f4f0d4693 introduced code to
change the types of SEL and MOV instructions for moves that simply
"copy bits around".  It didn't account for type conversion moves,
however.  So it would happily turn this:

   mov(8) vgrf6:D, -vgrf5:D
   mov(8) vgrf7:F, vgrf6:UD

into this:

   mov(8) vgrf6:D, -vgrf5:D
   mov(8) vgrf7:D, -vgrf5:D

which erroneously drops the conversion to float.

Cc: "11.0 10.6" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp