From 11c904d0d30c3b6507be7e3bece4677a40abbd63 Mon Sep 17 00:00:00 2001 From: Jose Maria Casanova Crespo Date: Mon, 9 Jul 2018 02:00:34 +0200 Subject: [PATCH] intel/compiler: relax brw_eu_validate for byte raw movs When the destination is a BYTE type allow raw movs even if the stride is not exact multiple of destination type and exec type, execution type is Word and its size is 2. This restriction was only allowing stride==2 destinations for 8-bit types. Reviewed-by: Jason Ekstrand --- src/intel/compiler/brw_eu_validate.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/intel/compiler/brw_eu_validate.c b/src/intel/compiler/brw_eu_validate.c index 29d1fe46f71..a25010b225c 100644 --- a/src/intel/compiler/brw_eu_validate.c +++ b/src/intel/compiler/brw_eu_validate.c @@ -472,9 +472,11 @@ general_restrictions_based_on_operand_types(const struct gen_device_info *devinf dst_type_size = 8; if (exec_type_size > dst_type_size) { - ERROR_IF(dst_stride * dst_type_size != exec_type_size, - "Destination stride must be equal to the ratio of the sizes of " - "the execution data type to the destination type"); + if (!(dst_type_is_byte && inst_is_raw_move(devinfo, inst))) { + ERROR_IF(dst_stride * dst_type_size != exec_type_size, + "Destination stride must be equal to the ratio of the sizes " + "of the execution data type to the destination type"); + } unsigned subreg = brw_inst_dst_da1_subreg_nr(devinfo, inst); -- 2.30.2