From e1afffe7fa7bd8e1cd1f7e58cfa2f33faf889628 Mon Sep 17 00:00:00 2001 From: Caio Oliveira Date: Thu, 7 Mar 2024 16:11:49 -0800 Subject: [PATCH] intel/brw: Use hstride instead of stride for accumulator The `stride` field is not meant to be used by ARFs (like the accumulator), and is always 1. Use the `hstride` instead. Reviewed-by: Kenneth Graunke Part-of: --- src/intel/compiler/brw_fs_lower_regioning.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/compiler/brw_fs_lower_regioning.cpp b/src/intel/compiler/brw_fs_lower_regioning.cpp index ed40d841b6d..cb125e197b5 100644 --- a/src/intel/compiler/brw_fs_lower_regioning.cpp +++ b/src/intel/compiler/brw_fs_lower_regioning.cpp @@ -66,7 +66,7 @@ namespace { * lowering pass will detect the mismatch in has_invalid_src_region * and fix the sources of the multiply instead of the destination. */ - return inst->dst.stride * type_sz(inst->dst.type); + return inst->dst.hstride * type_sz(inst->dst.type); } else if (type_sz(inst->dst.type) < get_exec_type_size(inst) && !is_byte_raw_mov(inst)) { return get_exec_type_size(inst);