i965/fs: Make emit_if_gen6 never fall back to emit_bool_to_cond_code.
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 4 Sep 2014 07:18:44 +0000 (00:18 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 8 Sep 2014 22:43:51 +0000 (15:43 -0700)
commitb9699e09bc2f4418093df00d212166c84e3cd71e
tree827c78282ac205e2b951d56fc589d70852d78918
parent6272e60ca394a8da178d3352831a48f4c429a3bc
i965/fs: Make emit_if_gen6 never fall back to emit_bool_to_cond_code.

Matt and I believe that Sandybridge actually uses 0xFFFFFFFF for a
"true" comparison result, similar to Ivybridge.  This matches the
internal documentation, and empirical results, but contradicts the PRM.

So, the comment is inaccurate, and we can actually just handle these
directly without ever needing to fall through to the condition code
path.

Also, the vec4 backend has always done it this way, and has apparently
been working fine.  This patch makes the FS backend match the vec4
backend's behavior.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp