i965/fs: Don't emit unnecessary SEL instruction from emit_image_atomic().

The SEL instruction with predication mode NONE emitted when the atomic
operation doesn't need to be predicated is a no-op and might rely on
undocumented hardware behaviour.  Noticed by chance while looking at
the assembly output.

Reviewed-by: Matt Turner <mattst88@gmail.com>
This commit is contained in:
Francisco Jerez 2016-02-06 18:43:45 -08:00
parent c300559fbf
commit 1817e3c07a
1 changed files with 1 additions and 1 deletions

View File

@ -1122,7 +1122,7 @@ namespace brw {
dims, rsize, op, pred);
/* An unbound surface access should give zero as result. */
if (rsize)
if (rsize && pred)
set_predicate(pred, bld.SEL(tmp, tmp, brw_imm_d(0)));
return tmp;