i965: Fix constant propagation into 32-bit integer MUL.
authorPaul Berry <stereotype441@gmail.com>
Wed, 2 Nov 2011 03:35:23 +0000 (20:35 -0700)
committerPaul Berry <stereotype441@gmail.com>
Fri, 4 Nov 2011 01:18:34 +0000 (18:18 -0700)
commit8fad0f99989866eeb72889a84f12f6f817334ddb
treee67e33cabcf02d607eacb5b556e23de723ba912b
parentdf73a70fbab40fe18e127bc614a9f4cb9e5ed006
i965: Fix constant propagation into 32-bit integer MUL.

i965's MUL instruction can't take an immediate value as its first
argument.  So normally, if constant propagation wants to propagate a
constant into the first argument of a MUL instruction, it swaps the
order of the two arguments.

This doesn't work for 32-bit integer (and unsigned integer)
multiplies, because the MUL operation is asymmetric in that case (it
multiplies 16 bits of one operand by 32 bits of the other).

Fixes piglit tests {vs,fs}-multiply-const-{ivec4,uvec4}.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/brw_fs.cpp
src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp