diff options
| author | Boris Brezillon <boris.brezillon@collabora.com> | 2021-01-13 11:18:11 +0100 |
|---|---|---|
| committer | Marge Bot <eric+marge@anholt.net> | 2021-01-13 17:07:14 +0000 |
| commit | 0ad83e33619a1d7547c34972eb1c6df62ed0bce7 (patch) | |
| tree | 07c0579e68387e3d8439e7296d99bf43240f38ae /src | |
| parent | f9237619d353d34c6d471cd39fe31c765d8dc529 (diff) | |
pan/bi: Fix the !immediate case in bi_emit_store_vary()
The base offset was ignored, take it into account.
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8469>
Diffstat (limited to 'src')
| -rw-r--r-- | src/panfrost/bifrost/bifrost_compile.c | 27 |
1 files changed, 17 insertions, 10 deletions
diff --git a/src/panfrost/bifrost/bifrost_compile.c b/src/panfrost/bifrost/bifrost_compile.c index 9405547bcb8..2e9c42a3efb 100644 --- a/src/panfrost/bifrost/bifrost_compile.c +++ b/src/panfrost/bifrost/bifrost_compile.c @@ -445,19 +445,26 @@ bi_emit_store_vary(bi_builder *b, nir_intrinsic_instr *instr) nir_alu_type T = nir_intrinsic_src_type(instr); enum bi_register_format regfmt = bi_reg_fmt_for_nir(T); - nir_src *offset = nir_get_io_offset_src(instr); unsigned imm_index = 0; bool immediate = bi_is_intr_immediate(instr, &imm_index, 16); - bi_index address = immediate ? - bi_lea_attr_imm(b, - bi_register(61), /* TODO RA */ - bi_register(62), /* TODO RA */ - regfmt, imm_index) : - bi_lea_attr(b, - bi_register(61), /* TODO RA */ - bi_register(62), /* TODO RA */ - bi_src_index(offset), regfmt); + bi_index address; + if (immediate) { + address = bi_lea_attr_imm(b, + bi_register(61), /* TODO RA */ + bi_register(62), /* TODO RA */ + regfmt, imm_index); + } else { + bi_index idx = + bi_iadd_u32(b, + bi_src_index(nir_get_io_offset_src(instr)), + bi_imm_u32(nir_intrinsic_base(instr)), + false); + address = bi_lea_attr(b, + bi_register(61), /* TODO RA */ + bi_register(62), /* TODO RA */ + idx, regfmt); + } /* Only look at the total components needed. In effect, we fill in all * the intermediate "holes" in the write mask, since we can't mask off |