[Mesa-dev] [PATCH 2/2] intel/fs: Initialize fs_visitor::grf_used on construction.
Francisco Jerez
currojerez at riseup.net
Mon Dec 18 23:26:53 UTC 2017
This should shut up some Valgrind errors during pre-regalloc
scheduling. The errors were harmless since they could only have led
to the estimation of the bank conflict penalty of an instruction
pre-regalloc, which is inaccurate at that point of the program
compilation, but no less accurate than the intended "return 0"
fall-back path. The scheduling pass is normally re-run after regalloc
with a well-defined grf_used value and accurate bank conflict
information.
Fixes: acf98ff933d "intel/fs: Teach instruction scheduler about GRF bank conflict cycles."
Reported-by: Eero Tamminen <eero.t.tamminen at intel.com>
---
src/intel/compiler/brw_fs_visitor.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/intel/compiler/brw_fs_visitor.cpp b/src/intel/compiler/brw_fs_visitor.cpp
index 481d9c51e7a..7a5f6451f2b 100644
--- a/src/intel/compiler/brw_fs_visitor.cpp
+++ b/src/intel/compiler/brw_fs_visitor.cpp
@@ -898,6 +898,7 @@ fs_visitor::init()
this->promoted_constants = 0,
+ this->grf_used = 0;
this->spilled_any_registers = false;
}
--
2.14.2
More information about the mesa-dev
mailing list