[Mesa-dev] [PATCH] i965/fs: don't do IR validation in release mode
Connor Abbott
cwabbott0 at gmail.com
Mon May 22 03:46:01 UTC 2017
Surprisingly, profiling shows that this takes quite a lot of time. So
turn it off when building in release mode, just like all the other
validation passes we do.
Difference at 95.0% confidence
-82.5956 +/- 11.46
-7.68394% +/- 1.06613%
(Student's t, pooled s = 15.8735)
Signed-off-by: Connor Abbott <cwabbott0 at gmail.com>
---
src/intel/compiler/brw_fs_validate.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/intel/compiler/brw_fs_validate.cpp b/src/intel/compiler/brw_fs_validate.cpp
index 676942c..cdccf0f 100644
--- a/src/intel/compiler/brw_fs_validate.cpp
+++ b/src/intel/compiler/brw_fs_validate.cpp
@@ -41,6 +41,7 @@
void
fs_visitor::validate()
{
+#ifdef DEBUG
foreach_block_and_inst (block, fs_inst, inst, cfg) {
if (inst->dst.file == VGRF) {
fsv_assert(inst->dst.offset / REG_SIZE + regs_written(inst) <=
@@ -54,4 +55,5 @@ fs_visitor::validate()
}
}
}
+#endif
}
--
2.9.3
More information about the mesa-dev
mailing list