nir/nir_opt_offsets: Prevent offsets going above max
In try_fold_load_store when trying to extract const addition from
non-const offset source, we should take into account that there is
already a constant base offset, which should count towards the limit.
The issue was found in "Monster Hunter: World" running on Turnip.
Fixes: cac6f633b2
("nir/opt_offsets: Use nir_ssa_scalar to chase offset additions.")
Well, the issue was present before this commit but it made a lot
of changes in surrounding code.
Signed-off-by: Danylo Piliaiev <dpiliaiev@igalia.com>
Reviewed-by: Timur Kristóf <timur.kristof@gmail.com
Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20099>
This commit is contained in:
committed by
Marge Bot
parent
f7e76eee28
commit
5d025f4003
@@ -123,7 +123,7 @@ try_fold_load_store(nir_builder *b,
|
||||
if (!nir_src_is_const(*off_src)) {
|
||||
uint32_t add_offset = 0;
|
||||
nir_ssa_scalar val = {.def = off_src->ssa, .comp = 0};
|
||||
val = try_extract_const_addition(b, val, state, &add_offset, max);
|
||||
val = try_extract_const_addition(b, val, state, &add_offset, max - off_const);
|
||||
if (add_offset == 0)
|
||||
return false;
|
||||
off_const += add_offset;
|
||||
@@ -139,6 +139,8 @@ try_fold_load_store(nir_builder *b,
|
||||
return false;
|
||||
|
||||
nir_instr_rewrite_src(&intrin->instr, &intrin->src[offset_src_idx], nir_src_for_ssa(replace_src));
|
||||
|
||||
assert(off_const <= max);
|
||||
nir_intrinsic_set_base(intrin, off_const);
|
||||
return true;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user