NeroReflex/mesa
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

aco/ra: don't initialize assigned in initializer list

Browse source 
According to Valgrind, vcc/m0 are uninitialized and this fixes it.

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27583>
(cherry picked from commit 53800191a3a7f38e2211768d7f75ebbf5508bf69)
This commit is contained in:
Rhys Perry 2024-02-12 16:38:28 +00:00 • committed by Eric Engestrom
parent db5a997626
commit c7f3e736a0
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
.pick_status.json
View file

@ -1984,7 +1984,7 @@
"description": "aco/ra: don't initialize assigned in initializer list", "description": "aco/ra: don't initialize assigned in initializer list",
"nominated": true, "nominated": true,
"nomination_type": 0, "nomination_type": 0,
"resolution": 0, "resolution": 1,
"main_sha": null, "main_sha": null,
"because_sha": null, "because_sha": null,
"notes": null "notes": null

2
src/amd/compiler/aco_register_allocation.cpp
View file

@ -62,7 +62,7 @@ struct assignment {
}; };
uint32_t affinity = 0; uint32_t affinity = 0;
assignment() = default; assignment() = default;
assignment(PhysReg reg_, RegClass rc_) : reg(reg_), rc(rc_), assigned(-1) {} assignment(PhysReg reg_, RegClass rc_) : reg(reg_), rc(rc_) { assigned = true; }
void set(const Definition& def) void set(const Definition& def)
{ {
assigned = true; assigned = true;