From: Rafael Antognolli Date: Fri, 19 Oct 2018 22:33:50 +0000 (-0700) Subject: intel/fs: Only propagate saturation if exec_size is the same. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0778748ebadca7991c541ec674024c2bec5f08f5;p=mesa.git intel/fs: Only propagate saturation if exec_size is the same. Otherwise it could propagate the saturation from a SIMD16 instruction into a SIMD8 instruction. With that, only part of the destination register, which is the source of the move with saturation, would have been updated. Reviewed-by: Matt Turner --- diff --git a/src/intel/compiler/brw_fs_saturate_propagation.cpp b/src/intel/compiler/brw_fs_saturate_propagation.cpp index 1e1461063ae..fe3fa7ecfea 100644 --- a/src/intel/compiler/brw_fs_saturate_propagation.cpp +++ b/src/intel/compiler/brw_fs_saturate_propagation.cpp @@ -65,7 +65,8 @@ opt_saturate_propagation_local(fs_visitor *v, bblock_t *block, bool interfered = false; foreach_inst_in_block_reverse_starting_from(fs_inst, scan_inst, inst) { - if (regions_overlap(scan_inst->dst, scan_inst->size_written, + if (scan_inst->exec_size == inst->exec_size && + regions_overlap(scan_inst->dst, scan_inst->size_written, inst->src[0], inst->size_read(0))) { if (scan_inst->is_partial_var_write(dispatch_width) || (scan_inst->dst.type != inst->dst.type &&