i965/fs: Apply gl_FrontFacing ? -1 : 1 optimization only for floats
authorIan Romanick <ian.d.romanick@intel.com>
Sat, 28 Feb 2015 16:32:57 +0000 (08:32 -0800)
committerIan Romanick <ian.d.romanick@intel.com>
Tue, 17 Mar 2015 22:01:44 +0000 (15:01 -0700)
commit6db5e134b627d24e3d6d42a6835e6595652c5aab
tree1323e11315faba83342082489d1d9dfa0f4e050c
parent4a53445b0d199489b2d1ae7d8654791e42b16804
i965/fs: Apply gl_FrontFacing ? -1 : 1 optimization only for floats

At the very least, unreal4/sun-temple/102.shader_test uses this pattern
for a signed integer result.  However, that shader did not hit the
optimization in the first place because it uses !gl_FrontFacing.  I
changed the shader to use remove the logical-not and reverse the other
operands.  I verified that incorrect code is generated before this
change and correct code is generated after.

Fixes fs-frontfacing-ternary-1-neg-1.shader_test.

No shader-db changes.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp