i965/fs: Detect multi-register MOVs correctly in register_coalesce.
authorFrancisco Jerez <currojerez@riseup.net>
Tue, 28 Jul 2015 08:25:59 +0000 (11:25 +0300)
committerFrancisco Jerez <currojerez@riseup.net>
Wed, 29 Jul 2015 10:27:12 +0000 (13:27 +0300)
register_coalesce() was considering the exec_size of the MOV
instruction alone to decide whether the register at offset+1 of the
source VGRF was being copied to inst->dst.reg_offset+1 of the
destination VGRF, which is only a valid assumption if the move has a
32-bit execution type.  Use regs_read() instead to find out the number
of registers copied by the instruction.

Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp

index a253c81c845c0d1c7fa4203b673b9fc3aa07db79..4f00b7f146ff0810a5235a9f902b37b8549ed68c 100644 (file)
@@ -195,7 +195,7 @@ fs_visitor::register_coalesce()
             continue;
          }
          reg_to_offset[offset] = inst->dst.reg_offset;
-         if (inst->exec_size == 16)
+         if (inst->regs_written > 1)
             reg_to_offset[offset + 1] = inst->dst.reg_offset + 1;
          mov[offset] = inst;
          channels_remaining -= inst->regs_written;