diff options
| author | Rafael Antognolli <[email protected]> | 2018-10-19 15:33:50 -0700 |
|---|---|---|
| committer | Rafael Antognolli <[email protected]> | 2019-04-22 16:53:55 -0700 |
| commit | 0778748ebadca7991c541ec674024c2bec5f08f5 (patch) | |
| tree | 3ec2a5a4f9870a47c62f9ab8e30b314ec7dab0db | |
| parent | 087f92c59a4d169f3912c621ea5a573a838d1be3 (diff) | |
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 <[email protected]>
| -rw-r--r-- | src/intel/compiler/brw_fs_saturate_propagation.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
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 && |