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

aco: use a temporary SGPR for 8-bit/16-bit literal reduction identities

Browse source 
Otherwise, the compiler overwrites s0 which contains the exec mask.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4494>
This commit is contained in:
Samuel Pitoiset 2020-05-18 16:00:10 +02:00 • committed by Marge Bot
parent b3c87c52ea
commit c76595aec2
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/amd/compiler/aco_reduce_assign.cpp
View file

@ -163,9 +163,11 @@ void setup_reduce_temp(Program* program)
bool need_sitmp = (program->chip_class <= GFX7 || program->chip_class >= GFX10) && instr->opcode != aco_opcode::p_reduce;
if (instr->opcode == aco_opcode::p_exclusive_scan) {
need_sitmp |=
(op == imin32 || op == imin64 || op == imax32 || op == imax64 ||
op == fmin32 || op == fmin64 || op == fmax32 || op == fmax64 ||
op == fmul64);
(op == imin8 || op == imin16 || op == imin32 || op == imin64 ||
op == imax8 || op == imax16 || op == imax32 || op == imax64 ||
op == fmin16 || op == fmin32 || op == fmin64 ||
op == fmax16 || op == fmax32 || op == fmax64 ||
op == fmul16 || op == fmul64);
}
if (need_sitmp) {
instr->definitions[2] = bld.def(RegClass(RegType::sgpr, instr->operands[0].size()));