Mesa (master): aco: add a few missing checks in value numbering
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Tue Oct 22 19:56:11 UTC 2019
Module: Mesa
Branch: master
Commit: 57c2cfb6080bce2222ef5f5ca72c444e878411bc
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=57c2cfb6080bce2222ef5f5ca72c444e878411bc
Author: Rhys Perry <pendingchaos02 at gmail.com>
Date: Mon Oct 14 17:19:19 2019 +0100
aco: add a few missing checks in value numbering
Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
Reviewed-by: Daniel Schürmann <daniel at schuermann.dev>
---
src/amd/compiler/aco_opt_value_numbering.cpp | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/amd/compiler/aco_opt_value_numbering.cpp b/src/amd/compiler/aco_opt_value_numbering.cpp
index e19f125e29f..fe094ebb219 100644
--- a/src/amd/compiler/aco_opt_value_numbering.cpp
+++ b/src/amd/compiler/aco_opt_value_numbering.cpp
@@ -191,7 +191,9 @@ struct InstrPred {
/* this is fine since they are only used for vertex input fetches */
MTBUF_instruction* aM = static_cast<MTBUF_instruction *>(a);
MTBUF_instruction* bM = static_cast<MTBUF_instruction *>(b);
- return aM->dfmt == bM->dfmt &&
+ return aM->can_reorder == bM->can_reorder &&
+ aM->barrier == bM->barrier &&
+ aM->dfmt == bM->dfmt &&
aM->nfmt == bM->nfmt &&
aM->offset == bM->offset &&
aM->offen == bM->offen &&
@@ -221,6 +223,7 @@ struct InstrPred {
MIMG_instruction* aM = static_cast<MIMG_instruction*>(a);
MIMG_instruction* bM = static_cast<MIMG_instruction*>(b);
return aM->can_reorder && bM->can_reorder &&
+ aM->barrier == bM->barrier &&
aM->dmask == bM->dmask &&
aM->unrm == bM->unrm &&
aM->glc == bM->glc &&
More information about the mesa-commit
mailing list