Mesa (staging/22.1): aco/ra: update register file when updating phi definition
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Jul 6 17:04:12 UTC 2022
Module: Mesa
Branch: staging/22.1
Commit: a9b3df51e026a4c5691cbd431d081ed9c7d16fe5
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a9b3df51e026a4c5691cbd431d081ed9c7d16fe5
Author: Rhys Perry <pendingchaos02 at gmail.com>
Date: Wed Jun 29 14:13:38 2022 +0100
aco/ra: update register file when updating phi definition
update_renames() fills in the wrong temp id.
Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
Reviewed-by: Daniel Schürmann <daniel at schuermann.dev>
Fixes: 302cb5c9001 ("aco/ra: remove some redundant code")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17295>
(cherry picked from commit 84f04fd0800384306347277c4e39524f0b295e34)
---
.pick_status.json | 2 +-
src/amd/compiler/aco_register_allocation.cpp | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/.pick_status.json b/.pick_status.json
index 5fe4aabace5..e4d19ab376b 100644
--- a/.pick_status.json
+++ b/.pick_status.json
@@ -1282,7 +1282,7 @@
"description": "aco/ra: update register file when updating phi definition",
"nominated": true,
"nomination_type": 1,
- "resolution": 0,
+ "resolution": 1,
"main_sha": null,
"because_sha": "302cb5c90015e2d3e139d575b3c7165519a21228"
},
diff --git a/src/amd/compiler/aco_register_allocation.cpp b/src/amd/compiler/aco_register_allocation.cpp
index 1f1b52e1e22..589f28144d7 100644
--- a/src/amd/compiler/aco_register_allocation.cpp
+++ b/src/amd/compiler/aco_register_allocation.cpp
@@ -2012,6 +2012,7 @@ get_reg_phi(ra_ctx& ctx, IDSet& live_in, RegisterFile& register_file,
if (prev_phi) {
/* if so, just update that phi's register */
prev_phi->definitions[0].setFixed(pc.second.physReg());
+ register_file.fill(prev_phi->definitions[0]);
ctx.assignments[prev_phi->definitions[0].tempId()] = {pc.second.physReg(),
pc.second.regClass()};
continue;
More information about the mesa-commit
mailing list