i965/fs: Fix off-by-one region overlap comparison in copy propagation.
authorFrancisco Jerez <currojerez@riseup.net>
Tue, 24 May 2016 02:32:51 +0000 (19:32 -0700)
committerFrancisco Jerez <currojerez@riseup.net>
Sat, 28 May 2016 06:19:20 +0000 (23:19 -0700)
commit2db9dd5aeb9566c8480651989981cb1169957748
treee253f299e5716532b6afc2a2566d21574c81cc3c
parent8f538d9ae0e09da0a48233332dd3a26de863069d
i965/fs: Fix off-by-one region overlap comparison in copy propagation.

This was introduced in cf375a3333e54a01462f192202d609436e5fbec8 but
the blame is mine because the pseudocode I sent in my review comment
for the original patch suggesting to do things this way already had
the off-by-one error.  This may have caused copy propagation to be
unnecessarily strict while checking whether VGRF writes interfere with
any ACP entries and possibly miss valid optimization opportunities in
cases where multiple copy instructions write sequential locations of
the same VGRF.

Cc: Iago Toral Quiroga <itoral@igalia.com>
Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp