fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-03-29 07:10:34 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

intel/eu/gen7: Fix brw_MOV() with DF destination and strided source.

Browse source 
I triggered this bug while prototyping code for a future platform on
IVB.  Could be a problem today though if a strided move is
copy-propagated into a type-converting move with DF destination.

Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
(cherry picked from commit 464e79144f)
This commit is contained in:
Francisco Jerez 2018-12-07 14:05:52 -08:00 • committed by Emil Velikov
parent 7d5057bfe4
commit c44c83ddd2
1 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/intel/compiler/brw_eu_emit.c
View file

@ -925,8 +925,8 @@ brw_MOV(struct brw_codegen *p, struct brw_reg dest, struct brw_reg src0)
const struct gen_device_info *devinfo = p->devinfo;
/* When converting F->DF on IVB/BYT, every odd source channel is ignored.
* To avoid the problems that causes, we use a <1,2,0> source region to read
* each element twice.
* To avoid the problems that causes, we use an <X,2,0> source region to
* read each element twice.
*/
if (devinfo->gen == 7 && !devinfo->is_haswell &&
brw_get_default_access_mode(p) == BRW_ALIGN_1 &&
@ -935,11 +935,8 @@ brw_MOV(struct brw_codegen *p, struct brw_reg dest, struct brw_reg src0)
src0.type == BRW_REGISTER_TYPE_D ||
src0.type == BRW_REGISTER_TYPE_UD) &&
!has_scalar_region(src0)) {
assert(src0.vstride == BRW_VERTICAL_STRIDE_4 &&
src0.width == BRW_WIDTH_4 &&
src0.hstride == BRW_HORIZONTAL_STRIDE_1);
src0.vstride = BRW_VERTICAL_STRIDE_1;
assert(src0.vstride == src0.width + src0.hstride);
src0.vstride = src0.hstride;
src0.width = BRW_WIDTH_2;
src0.hstride = BRW_HORIZONTAL_STRIDE_0;
}