[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