fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-05 07:28:11 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

r600/sfn: make sure we return a non-negative number of registers

Browse source 
If a shader doesn't use any register and only ssa values we might
end up with zero minimum registers, and because a unsigned is
returned that goes wrong.

Fixes: 565816dfa15214abbeef9a9d94e44f30507ca4d7
    r600/sfn: Set minimum required registers based on array allocation

Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/8008

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20516>
This commit is contained in:
Gert Wollny 2023-01-04 14:31:48 +01:00 • committed by Marge Bot
parent c3e2e71b50
commit 234e8fd157
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/gallium/drivers/r600/sfn/sfn_shader.cpp
View file

@ -520,7 +520,8 @@ Shader::allocate_local_registers(const exec_list *registers)
{
if (value_factory().allocate_registers(registers))
m_indirect_files |= 1 << TGSI_FILE_TEMPORARY;
m_required_registers = value_factory().next_register_index() - 1;
m_required_registers = value_factory().next_register_index() ?
value_factory().next_register_index() - 1 : 0;
}
bool